quilt
quilt copied to clipboard
Make JsonDisplay nested keys lighter
Before:
After:
Codecov Report
Merging #3313 (f360020) into master (d3c490b) will increase coverage by
0.00%
. The diff coverage is66.66%
.
@@ Coverage Diff @@
## master #3313 +/- ##
=======================================
Coverage 35.18% 35.19%
=======================================
Files 674 674
Lines 29331 29337 +6
Branches 4336 4340 +4
=======================================
+ Hits 10321 10325 +4
- Misses 17835 17837 +2
Partials 1175 1175
Flag | Coverage Δ | |
---|---|---|
api-python | 90.79% <ø> (ø) |
|
catalog | 8.84% <66.66%> (+0.01%) |
:arrow_up: |
lambda | 86.05% <ø> (ø) |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
catalog/app/components/JsonDisplay/JsonDisplay.js | 0.00% <0.00%> (ø) |
|
catalog/app/utils/JSONOneliner.ts | 87.60% <100.00%> (+0.42%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
I think it was hard to read the one-liner, when it's not obvious which key are from the first level, and which from the second
Maybe, I'll wait feedback from customer who wanted these key/values (visually separated) in packages list