vorburger commented on pull request #857: URL: https://github.com/apache/fineract/pull/857#issuecomment-629656112
@ptuomola I'm totally with you on the principle that in theory we should never touch existing Flyway SQL migration scripts, only write new ones (that's kind of the point of Flyway). I suspect that you are correct that this will not give us automagic migration, but at least the script is cleaner for new installs? In this particular instance, it's probably "not worth it" to do the same via migration scripts - what do you think? Personally I don't really have a problem with this getting merged. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
