Skip to content

fix compose config --format json#11630

Merged
glours merged 1 commit intodocker:mainfrom
ndeloof:config_json
Mar 18, 2024
Merged

fix compose config --format json#11630
glours merged 1 commit intodocker:mainfrom
ndeloof:config_json

Conversation

@ndeloof
Copy link
Copy Markdown
Contributor

@ndeloof ndeloof commented Mar 18, 2024

What I did
fix regression generating json output for config
also introduce e2e test for this feature which deserve some love

Related issue
fixes #11627

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

Signed-off-by: Nicolas De Loof <[email protected]>
@ndeloof ndeloof requested review from a team, glours, jhrotko and milas and removed request for a team March 18, 2024 07:10
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 18, 2024

Codecov Report

Attention: Patch coverage is 71.42857% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 57.73%. Comparing base (f937e42) to head (bc5fc6b).

Files Patch % Lines
cmd/compose/config.go 71.42% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11630      +/-   ##
==========================================
- Coverage   57.77%   57.73%   -0.04%     
==========================================
  Files         138      138              
  Lines       11730    11736       +6     
==========================================
- Hits         6777     6776       -1     
- Misses       4278     4284       +6     
- Partials      675      676       +1     

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

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] config command format JSON broken

2 participants