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

use compose-go version fixing extra_hosts unicity issue #12159

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

glours
Copy link
Contributor

@glours glours commented Sep 24, 2024

What I did
Bump compose-go version fixing extra_hosts unicity issue

Related issue
fixes #12155

(not mandatory) A picture of a cute animal, if possible in relation to what you did

@glours glours self-assigned this Sep 24, 2024
@glours glours requested review from a team, ndeloof and jhrotko and removed request for a team September 24, 2024 13:01
go.mod Outdated
@@ -192,3 +192,5 @@ require (
sigs.k8s.io/yaml v1.3.0 // indirect
tags.cncf.io/container-device-interface v0.8.0 // indirect
)

replace github.com/compose-spec/compose-go/v2 => github.com/glours/compose-go/v2 v2.0.0-20240924124937-38f94a79402c
Copy link
Contributor

Choose a reason for hiding this comment

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

?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just to test again Compose CI until we release a new version of compose-go

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I marked the PR as draft for now

Copy link
Contributor

Choose a reason for hiding this comment

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

right, I got confused for this reason as I got notified on this one first :)

@glours glours marked this pull request as draft September 24, 2024 13:08
@glours glours force-pushed the fix-extra_hosts_unicity-issue branch 2 times, most recently from 697c12b to 2ba208e Compare September 26, 2024 09:32
@glours glours force-pushed the fix-extra_hosts_unicity-issue branch from 2ba208e to 9a7a95f Compare September 26, 2024 14:14
@glours glours requested a review from ndeloof September 26, 2024 14:17
@glours glours marked this pull request as ready for review September 26, 2024 14:17
@jhrotko jhrotko merged commit c9d96b4 into docker:main Sep 26, 2024
28 checks passed
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.

[BUG] <extra_hosts array items[0,1] must be unique>
3 participants