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

Update Docker command in CI to work with current syntax #647

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

vtnate
Copy link
Contributor

@vtnate vtnate commented Aug 7, 2024

Any background context you want to provide?

On July 29 Github Actions removed access to Docker Compose v1 (which has been deprecated since July 2023). This means we have to use the v2 syntax, which this PR achieves.

Since we have this in an early step of our CI workflow, all CI will fail until this is merged.

What does this PR accomplish?

Use Docker Compose v2 syntax in a CI check

How should this be manually tested?

CI is sufficient

@vtnate vtnate added the ignore-for-release Exclude this PR from release notes label Aug 7, 2024
@vtnate vtnate self-assigned this Aug 7, 2024
@nllong nllong merged commit 28a458c into develop Aug 9, 2024
10 checks passed
@nllong nllong deleted the fix-CI-docker branch August 9, 2024 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release Exclude this PR from release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants