Issue 477 in flyway: Fail fast for missing placeholders

4 views
Skip to first unread message

fly...@googlecode.com

unread,
Apr 10, 2013, 10:07:11 AM4/10/13
to flywa...@googlegroups.com
Status: New
Owner: ----
Labels: Type-Enhancement Priority-Medium

New issue 477 by timan...@gmail.com: Fail fast for missing placeholders
http://code.google.com/p/flyway/issues/detail?id=477

If there are placeholders in the migration scripts that Flyway is running,
it should fail with a clear error message about the missing placeholder
rather than use the literal text of the placeholder variable. This
enhancement would to reduce the time for troubleshooting missing
configuration.

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

fly...@googlecode.com

unread,
Apr 20, 2013, 5:55:44 AM4/20/13
to flywa...@googlegroups.com
Updates:
Status: Accepted
Labels: Component-Core

Comment #1 on issue 477 by axel.fontai...@gmail.com: Fail fast for
missing placeholders
http://code.google.com/p/flyway/issues/detail?id=477

Hi Tim,

good suggestion! I'll look into this.

Cheers
Axel

fly...@googlecode.com

unread,
May 8, 2013, 6:51:22 PM5/8/13
to flywa...@googlegroups.com

Comment #2 on issue 477 by brian.ha...@gmail.com: Fail fast for missing
placeholders
http://code.google.com/p/flyway/issues/detail?id=477

I am working on this on my fork: https://github.com/bjhartin/flyway.

fly...@googlecode.com

unread,
May 9, 2013, 5:11:17 PM5/9/13
to flywa...@googlegroups.com

Comment #3 on issue 477 by brian.ha...@gmail.com: Fail fast for missing
placeholders
http://code.google.com/p/flyway/issues/detail?id=477

Pull request submitted.

fly...@googlecode.com

unread,
May 16, 2013, 12:36:18 PM5/16/13
to flywa...@googlegroups.com

Comment #4 on issue 477 by brian.ha...@gmail.com: Fail fast for missing
placeholders
http://code.google.com/p/flyway/issues/detail?id=477

Axel,

Did you have any feedback about these changes? We are anxious to have this
feature in an upcoming release.

Thank you,

Brian

fly...@googlecode.com

unread,
May 24, 2013, 11:16:42 AM5/24/13
to flywa...@googlegroups.com
Updates:
Status: Fixed

Comment #5 on issue 477 by axel.fontai...@gmail.com: Fail fast for
missing placeholders
http://code.google.com/p/flyway/issues/detail?id=477

Pull request merged :-)

Thanks!
Axel

fly...@googlecode.com

unread,
Jul 31, 2013, 6:48:58 PM7/31/13
to flywa...@googlegroups.com

Comment #6 on issue 477 by hv.vik...@gmail.com: Fail fast for missing
placeholders
http://code.google.com/p/flyway/issues/detail?id=477

Is this feature configurable? Can this feature be turned off?
Reply all
Reply to author
Forward
0 new messages