-
Notifications
You must be signed in to change notification settings - Fork 0
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
Bump flyway #30
Bump flyway #30
Conversation
10990e9
to
f978fd6
Compare
f978fd6
to
75da994
Compare
db/flyway-clean-schema.sh
Outdated
@@ -2,5 +2,5 @@ | |||
|
|||
# Copyright 2023 Adevinta | |||
|
|||
docker run --net=host flyway/flyway:"${FLYWAY_VERSION:-9.19.4}-alpine" -community \ | |||
docker run --net=host flyway/flyway:"${FLYWAY_VERSION:-9.19.4}-alpine" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why should the old version continue to be used as fallback?
db/flyway-migrate.sh
Outdated
@@ -9,5 +9,5 @@ docker exec vultrackerdb psql -c "ALTER USER vultrackerdb_test WITH SUPERUSER;" | |||
docker exec vultrackerdb psql -c "CREATE DATABASE vultrackerdb_test;" -U vultrackerdb | |||
|
|||
|
|||
docker run --net=host --rm -v "$PWD"/sql:/flyway/sql flyway/flyway:"${FLYWAY_VERSION:-9.19.4}-alpine" -community \ | |||
docker run --net=host --rm -v "$PWD"/sql:/flyway/sql flyway/flyway:"${FLYWAY_VERSION:-9.19.4}-alpine" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment as above
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.