Skip to content

push docker images when .sql files are updated#10368

Merged
poikilotherm merged 1 commit intodevelopfrom
10367-docker-sql
Mar 12, 2024
Merged

push docker images when .sql files are updated#10368
poikilotherm merged 1 commit intodevelopfrom
10367-docker-sql

Conversation

@pdurbin
Copy link
Copy Markdown
Member

@pdurbin pdurbin commented Mar 12, 2024

What this PR does / why we need it:

When we fix duplicate Flyway scripts, we want the change to be reflected in the Docker image.

Which issue(s) this PR closes:

Special notes for your reviewer:

See the issue for a link to the Slack conversation.

Suggestions on how to test this:

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

No.

Is there a release notes update needed for this change?:

No.

Additional documentation:

None.

When we fix duplicate Flyway scripts, we want the change to be
reflected in the Docker image.
@pdurbin pdurbin requested a review from poikilotherm March 12, 2024 18:23
Copy link
Copy Markdown
Contributor

@poikilotherm poikilotherm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Good catch!

@poikilotherm poikilotherm merged commit ec1b174 into develop Mar 12, 2024
@poikilotherm poikilotherm deleted the 10367-docker-sql branch March 12, 2024 19:52
@pdurbin pdurbin added this to the 6.2 milestone Mar 12, 2024
@cmbz cmbz added the Size: 3 A percentage of a sprint. 2.1 hours. label Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: 3 A percentage of a sprint. 2.1 hours.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docker images are not pushed when only a .sql file is updated

3 participants