search

psyinfra / HTCAnalyze

MIT License
0 stars 3 forks source link

HTCAnalyser->HTCAnalyze; and related renames #106

Closed aqw closed 4 years ago

aqw commented 4 years ago

This addresses #100, and applies the same treatment to the "Summariser" mode.

This PR is based on top of #102.

If this is accepted, the only remaining step is to rename the project in GitHub.

aqw commented 4 years ago

The codacy nits seem to be unrelated to my change, and were already present in the code.

mathisloevenich commented 4 years ago

I will make this fix asap.

aqw commented 4 years ago

@mathisloevenich This is fixed in this PR. ;-)

codecov-io commented 4 years ago

Codecov Report

Merging #106 into master will not change coverage. The diff coverage is 68.18%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   76.63%   76.63%           
=======================================
  Files           4        4           
  Lines        1083     1083           
  Branches      266      266           
=======================================
  Hits          830      830           
  Misses        173      173           
  Partials       80       80
Impacted Files Coverage Δ
htcanalyze/logvalidator.py 67.77% <ø> (ø)
setup.py 0.00% <0.00%> (ø)
htcanalyze/htcanalyze.py 77.29% <42.85%> (ø)
htcanalyze/main.py 78.70% <82.75%> (ø)

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 ca0e80f...4448221. Read the comment docs.

mathisloevenich commented 4 years ago

I fixed some minor Issues with that last commit. I need to merge the PR #99 first, to make Codacy run clear

aqw commented 4 years ago

Rebased on master.

aqw commented 4 years ago

99 looks to be mostly obsolete now, between the fixes in #102 and this PR (#106).