Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

& in comments should not trigger SQL*Plus placeholder replacement #2432

Closed
axelfontaine opened this issue Jul 8, 2019 · 3 comments
Closed

& in comments should not trigger SQL*Plus placeholder replacement #2432

axelfontaine opened this issue Jul 8, 2019 · 3 comments

Comments

@axelfontaine
Copy link
Contributor

@axelfontaine axelfontaine commented Jul 8, 2019

Example:

/* 1.01.01     08/01/07    XX                 Fixed ABC & DEF */
@MikielAgutu
Copy link
Contributor

@MikielAgutu MikielAgutu commented Jul 8, 2019

According to this Stack Overflow post perhaps it's as simple as using set define off

@matthew-sprague
Copy link

@matthew-sprague matthew-sprague commented Jul 8, 2019

while that is a workaround, comments should be entirely ignored for all flyway interactions

@MikielAgutu
Copy link
Contributor

@MikielAgutu MikielAgutu commented Jul 31, 2019

This looks quite tricky to fix. We can confirm set define off does offer a workaround. Therefore we're pushing this one back.

@MikielAgutu MikielAgutu removed this from the Flyway 6.0.0 milestone Jul 31, 2019
@MikielAgutu MikielAgutu added this to the Flyway 6.1.0 milestone Jul 31, 2019
@MikielAgutu MikielAgutu removed this from the Flyway 6.1.0 milestone Oct 16, 2019
@MikielAgutu MikielAgutu added this to the Flyway 6.0.7 milestone Oct 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants