aesharpe commented 2 years ago

Still need to test and create tests. This is just open so that folks can check-in on progress

codecov[bot] commented 2 years ago

Codecov Report

Merging #20 (51cf81e) into main (4c4f2c7) will decrease coverage by 0.3%. The diff coverage is 90.0%.

@@           Coverage Diff           @@
##            main     #20     +/-   ##
=======================================
- Coverage   52.1%   51.7%   -0.4%     
=======================================
  Files         17      17             
  Lines        485     479      -6     
=======================================
- Hits         253     248      -5     
+ Misses       232     231      -1

Impacted Files	Coverage Δ
src/pudl_zenodo_storage/cli.py	`0.0% <0.0%> (ø)`
...c/pudl_zenodo_storage/frictionless/contributors.py	`100.0% <ø> (ø)`
src/pudl_zenodo_storage/frictionless/eia860.py	`100.0% <ø> (ø)`
src/pudl_zenodo_storage/frictionless/eia860m.py	`100.0% <ø> (ø)`
src/pudl_zenodo_storage/frictionless/eia923.py	`100.0% <ø> (ø)`
src/pudl_zenodo_storage/frictionless/ferc1.py	`100.0% <ø> (ø)`
src/pudl_zenodo_storage/frictionless/ferc2.py	`75.0% <ø> (-5.0%)`	:arrow_down:
src/pudl_zenodo_storage/frictionless/licenses.py	`100.0% <ø> (ø)`
src/pudl_zenodo_storage/__init__.py	`100.0% <100.0%> (ø)`
...frictionless/epacems_unitid_eia_plant_crosswalk.py	`100.0% <100.0%> (ø)`
... and 10 more

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 4c4f2c7...51cf81e. Read the comment docs.

aesharpe commented 2 years ago

Most of these changes are docs stuff. Lmk if you think I should change the deposition name to one of the other ones I suggested above!

aesharpe commented 2 years ago

Ok the most recent changes include a function that alters pudl contributor metadata to meet the zenodo contributor requirements.

catalyst-cooperative / pudl-zenodo-storage

archive epa eia crosswalk #20

Codecov Report