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

Create CSV artifact including json keys as top level fields #88

Merged
merged 9 commits into from
Apr 6, 2022

Conversation

Staberinde
Copy link
Contributor

@Staberinde Staberinde commented Apr 1, 2022

@codecov-commenter
Copy link

codecov-commenter commented Apr 1, 2022

Codecov Report

Merging #88 (27ae56d) into main (b964f64) will increase coverage by 0.25%.
The diff coverage is 85.18%.

@@            Coverage Diff             @@
##             main      #88      +/-   ##
==========================================
+ Coverage   68.10%   68.35%   +0.25%     
==========================================
  Files          60       60              
  Lines        2580     2604      +24     
==========================================
+ Hits         1757     1780      +23     
- Misses        823      824       +1     
Impacted Files Coverage Δ
digital_land/specification.py 91.72% <ø> (+1.99%) ⬆️
digital_land/cli.py 80.95% <50.00%> (-1.23%) ⬇️
digital_land/api.py 57.31% <95.23%> (+5.56%) ⬆️
digital_land/package/sqlite.py 60.64% <0.00%> (-0.65%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b964f64...27ae56d. Read the comment docs.

@Staberinde Staberinde changed the title Create CSV artifact including json keys as top level fields Create CSV artifact including json keys as top level fields [UNTESTED] Apr 1, 2022
@Staberinde Staberinde force-pushed the 2228-flatten-json-fields branch from 0b48816 to 3943496 Compare April 4, 2022 12:05
@Staberinde Staberinde changed the title Create CSV artifact including json keys as top level fields [UNTESTED] Create CSV artifact including json keys as top level fields Apr 4, 2022
@Staberinde Staberinde force-pushed the 2228-flatten-json-fields branch from d98504d to 28c58bd Compare April 6, 2022 15:20
@Staberinde Staberinde merged commit 3a47bea into main Apr 6, 2022
@ernest-tpximpact ernest-tpximpact deleted the 2228-flatten-json-fields branch September 22, 2023 12:57
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