Upgrade Flywaydb to v7.6.0#1974
Merged
wslulciuc merged 2 commits intoMay 3, 2022
Merged
Conversation
Signed-off-by: Dakshin Karthikeyan <dakshin.k1@gmail.com>
dakshin-k
force-pushed
the
dependency/org.flywaydb-flyway-core-7.6.0
branch
from
caee583 to
a821bd2
Compare
Codecov Report
@@ Coverage Diff @@
## main #1974 +/- ##
=========================================
Coverage 78.23% 78.23%
Complexity 955 955
=========================================
Files 194 194
Lines 5292 5293 +1
Branches 420 420
=========================================
+ Hits 4140 4141 +1
Misses 706 706
Partials 446 446
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
Contributor
Author
|
I'm not sure if this change needs an entry in |
wslulciuc
reviewed
May 3, 2022
|
|
||
| ### Changed | ||
|
|
||
| * Upgrade Flyway to v7.6.0 [@dakshin-k](https://github.com/dakshin-k) |
wslulciuc
approved these changes
May 3, 2022
wslulciuc
left a comment
wslulciuc
left a comment
Member
There was a problem hiding this comment.
Thanks, @dakshin-k for getting Marquez on the latest and greatest of flyway 💯 🥇
jonathanpmoraes
referenced
this pull request
in nubank/NuMarquez
Feb 6, 2025
Signed-off-by: Dakshin Karthikeyan <dakshin.k1@gmail.com> Co-authored-by: Dakshin Karthikeyan <dakshin.k1@gmail.com> Co-authored-by: Willy Lulciuc <willy@datakin.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Flywaydb changed the public migrate API to return an object instead of a raw integer. This updates the migration script to use the new API.
Closes: #1052
Solution
Incorporated the API change and bumped the Flyway version in
api/build.gradle.Checklist
CHANGELOG.mdwith details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary).sqldatabase schema migration according to Flyway's naming convention (if relevant)