Skip to content

Conversation

@seekskyworld
Copy link

@seekskyworld seekskyworld commented Jan 15, 2026

Issue

Summary

Motivation

  • keep compose file loading/processing in compose-go per maintainer guidance

Validation

  • go test ./... (fails locally: Windows path expectations + missing Docker config at temp HOME)

@seekskyworld seekskyworld requested a review from a team as a code owner January 15, 2026 14:23
@seekskyworld seekskyworld force-pushed the fix/12637-interpolation-warning branch from 1e5e887 to 75e1fd7 Compare January 15, 2026 14:36
Copy link
Contributor

@ndeloof ndeloof left a comment

Choose a reason for hiding this comment

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

compose file loading and processing should be implemented in compose-go

@seekskyworld seekskyworld force-pushed the fix/12637-interpolation-warning branch from 75e1fd7 to 463a5f1 Compare January 15, 2026 16:37
@seekskyworld
Copy link
Author

seekskyworld commented Jan 15, 2026

@ndeloof Updated the PR to align with your guidance: removed the compose-local workaround and now rely on the compose-go fix (compose-spec/compose-go#847) via a temporary go.mod replace. Once the compose-go PR lands, I can switch this to a clean version bump. Tests: go test ./... .

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.

2 participants