Skip to content

update compose-go to version using immutable Project functions#11307

Merged
glours merged 1 commit intodocker:mainfrom
glours:compose-go-immutable-projects
Jan 11, 2024
Merged

update compose-go to version using immutable Project functions#11307
glours merged 1 commit intodocker:mainfrom
glours:compose-go-immutable-projects

Conversation

@glours
Copy link
Contributor

@glours glours commented Dec 29, 2023

Before merging: need PR compose-spec/compose-go#518 to be merged and a new version of compose-go to be released

What I did
Test the version of compose-go which introduces Project immutable functions

  • replace compose-go with the version of the following PR
  • adapt current code to support the new compose-go version without using the advantage of the immutability for now, basically just replace the current Project instance with the new one.

Related issue
https://docker.atlassian.net/browse/COMP-380

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

@glours glours force-pushed the compose-go-immutable-projects branch 5 times, most recently from 2b15a47 to b466fa3 Compare December 29, 2023 14:58
@codecov
Copy link

codecov bot commented Dec 29, 2023

Codecov Report

Attention: 23 lines in your changes are missing coverage. Please review.

Comparison is base (6f62bcc) 56.69% compared to head (5d05df6) 56.67%.

Files Patch % Lines
pkg/compose/publish.go 0.00% 5 Missing ⚠️
cmd/compose/run.go 55.55% 4 Missing ⚠️
cmd/compose/config.go 0.00% 3 Missing ⚠️
cmd/compose/up.go 77.77% 2 Missing ⚠️
pkg/compose/compose.go 33.33% 2 Missing ⚠️
pkg/compose/watch.go 0.00% 2 Missing ⚠️
cmd/compose/pull.go 83.33% 1 Missing ⚠️
cmd/compose/push.go 0.00% 1 Missing ⚠️
cmd/compose/restart.go 0.00% 1 Missing ⚠️
cmd/compose/scale.go 0.00% 0 Missing and 1 partial ⚠️
... and 1 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11307      +/-   ##
==========================================
- Coverage   56.69%   56.67%   -0.02%     
==========================================
  Files         134      134              
  Lines       11448    11456       +8     
==========================================
+ Hits         6490     6493       +3     
- Misses       4336     4341       +5     
  Partials      622      622              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@glours glours self-assigned this Dec 29, 2023
@glours glours force-pushed the compose-go-immutable-projects branch 6 times, most recently from 85a15cd to 9e4fc83 Compare January 11, 2024 09:44
@glours glours marked this pull request as ready for review January 11, 2024 09:44
Signed-off-by: Guillaume Lours <705411+glours@users.noreply.github.com>
@glours glours force-pushed the compose-go-immutable-projects branch from 9e4fc83 to 5d05df6 Compare January 11, 2024 09:44
@glours glours requested review from jhrotko and milas January 11, 2024 09:54
@glours glours merged commit 8b16ab1 into docker:main Jan 11, 2024
@glours glours deleted the compose-go-immutable-projects branch June 30, 2025 21:22
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