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

fix: remove obsolete docker compose file version #2516

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

ghorbani-ali
Copy link
Contributor

Based on this reference, there is no need to compose file version.

@LaurentGoderre
Copy link
Member

THis file is autogenerated. Please modify it in this one: https://github.com/docker-library/docs/blob/master/wordpress/stack.yml#L1

@ghorbani-ali
Copy link
Contributor Author

Hi @LaurentGoderre , Thanks for the tip. I updated my PR.

@LaurentGoderre LaurentGoderre merged commit fe1476b into docker-library:master Dec 18, 2024
6 checks passed
@tianon
Copy link
Member

tianon commented Jan 6, 2025

This was still here because this is actually a Docker Swarm "Stack" file, from back when it and Compose were being co-designed intentionally. 🙈

I think this really ought to have been part of a larger change (like what's discussed/hinted at in #2514), but that's more complicated to orchestrate/coordinate. 😞 ❤️

(IMO, dropping version: in Compose was a mistake, because now compatibility is harder to break over time, but 🤷)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants