ArcanaFramework / arcana

Abstraction of Repository Centric ANAlysis
Other
4 stars 5 forks source link

Rework pipeline autodocs based on feedback #120 #129

Closed Prepultrue closed 2 years ago

Prepultrue commented 2 years ago

For #120:

Cleanup some pre-commit warnings on other areas of the codebase

Also includes changes from #88

codecov-commenter commented 2 years ago

Codecov Report

Merging #129 (92ed30f) into develop (4361561) will increase coverage by 0.12%. The diff coverage is 46.87%.

@@             Coverage Diff             @@
##           develop     #129      +/-   ##
===========================================
+ Coverage    76.77%   76.89%   +0.12%     
===========================================
  Files           57       57              
  Lines         5830     5830              
  Branches      1256     1258       +2     
===========================================
+ Hits          4476     4483       +7     
+ Misses        1032     1025       -7     
  Partials       322      322              
Impacted Files Coverage Δ
arcana/cli/apply.py 93.02% <ø> (-0.16%) :arrow_down:
arcana/cli/dataset.py 87.50% <ø> (ø)
arcana/cli/derive.py 56.66% <ø> (-0.72%) :arrow_down:
arcana/core/deploy/build.py 71.32% <ø> (ø)
arcana/core/deploy/utils.py 64.49% <0.00%> (-0.26%) :arrow_down:
arcana/data/stores/medimage/xnat/cs.py 80.89% <ø> (-0.42%) :arrow_down:
arcana/deploy/medimage/xnat.py 81.90% <ø> (ø)
arcana/test/fixtures/common.py 89.85% <ø> (-0.29%) :arrow_down:
arcana/test/fixtures/core.py 100.00% <ø> (ø)
arcana/test/fixtures/docs.py 100.00% <ø> (ø)
... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

tclose commented 2 years ago

Looks good, will merge when tests pass

tclose commented 2 years ago

@Prepultrue the docs build test is broken