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

Format inspect-builder json output before printing #953

Merged
merged 3 commits into from
Dec 3, 2020

Conversation

dwillist
Copy link
Contributor

Signed-off-by: dwillist dthornton@vmware.com

Summary

Simple formatting update for inspect-builder json output

Output

Output contents is equivalent json, just with some formatting for human readability.

Resolves #952

Signed-off-by: dwillist <dthornton@vmware.com>
@dwillist dwillist requested a review from a team as a code owner November 12, 2020 14:24
@codecov
Copy link

codecov bot commented Nov 12, 2020

Codecov Report

Merging #953 (15f0f8c) into main (12ad921) will decrease coverage by 0.02%.
The diff coverage is 71.43%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #953      +/-   ##
==========================================
- Coverage   77.70%   77.68%   -0.01%     
==========================================
  Files         103      103              
  Lines        5012     5017       +5     
==========================================
+ Hits         3894     3897       +3     
- Misses        724      725       +1     
- Partials      394      395       +1     
Flag Coverage Δ
os_linux 77.64% <71.43%> (+0.01%) ⬆️
os_macos 74.67% <71.43%> (-0.01%) ⬇️
os_windows 100.00% <ø> (ø)
unit 77.68% <71.43%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@dfreilich dfreilich added this to the 0.16.0 milestone Nov 18, 2020
@dfreilich dfreilich added the type/enhancement Issue that requests a new feature or improvement. label Nov 18, 2020
@dfreilich dfreilich changed the title format inspect-builder json output before printing Format inspect-builder json output before printing Nov 18, 2020
@dfreilich dfreilich merged commit 709c41b into buildpacks:main Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement Issue that requests a new feature or improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Beautify inspect-builder json output,
2 participants