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

Compose refactor #68503

Merged
merged 8 commits into from
Aug 15, 2024
Merged

Compose refactor #68503

merged 8 commits into from
Aug 15, 2024

Conversation

khawajaJunaid
Copy link
Contributor

@khawajaJunaid khawajaJunaid commented Aug 4, 2024

What?

removes 'version' attribute post deprecation from all compose files related to dev, staging and prod env

Why?

For the latest docker compose version i.e docker compose v2, the 'version' attribute has been deprecated, which is currently a warning when you do docker compose up but in future would become a requirement. https://github.com/compose-spec/compose-spec/blob/main/spec.md#version-and-name-top-level-elements this mentions that the version has become obsolete for the latest docker compose version.

How?

remove the version field in the compose files to remove the warning that one may get during builds
Closes NEXT-

@ijjk ijjk added the examples Issue/PR related to examples label Aug 4, 2024
@ijjk
Copy link
Member

ijjk commented Aug 4, 2024

Allow CI Workflow Run

  • approve CI run for commit: f0c9b4c

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@khawajaJunaid
Copy link
Contributor Author

hi @leerob, added the link to the documentation to the fact that the "version" attribute results in a warning as you asked, hope this helps

@leerob leerob merged commit 5223c92 into vercel:canary Aug 15, 2024
37 checks passed
@github-actions github-actions bot added the locked label Sep 1, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue/PR related to examples locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants