Skip to content

Fix CI: Use docker compose v2 syntax #669

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.

Sign up for GitHub

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

Jump to bottom
Open
jjroelofs wants to merge 1 commit into
base: 1.x
Choose a base branch
from
Open

Fix CI: Use docker compose v2 syntax #669

jjroelofs wants to merge 1 commit into from

Conversation

@jjroelofs
Copy link
Collaborator

@jjroelofs jjroelofs commented Nov 26, 2025

Linked issues

Solution

Update .github/workflows/review.yaml to use Docker Compose v2 syntax:

  • Change docker-compose to docker compose (v2 plugin, without hyphen)
  • Update actions/checkout from v2 to v4

GitHub Actions ubuntu-latest runners no longer include docker-compose v1 standalone binary, only the docker compose v2 plugin.

Checklist

  • I have read the CONTRIBUTING.md document.
  • My code follows the coding standards and style of this project.
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Need to run update.php after code changes
  • Requires a change to end-user documentation.
  • Requires a change to developer documentation.
  • Requires a change to QA tests.
  • Requires a new QA test.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.

Fix CI: Use docker compose v2 syntax
516d823 
GitHub Actions runners no longer include docker-compose v1 standalone
binary. Update to docker compose v2 plugin syntax (without hyphen).

Also update actions/checkout from v2 to v4.

Fixes #668
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jjroelofs