Closed paudetseis closed 3 years ago
Merging #20 (d6814d0) into master (2b77001) will decrease coverage by
2.76%
. The diff coverage is1.26%
.
@@ Coverage Diff @@
## master #20 +/- ##
=========================================
- Coverage 6.46% 3.69% -2.77%
=========================================
Files 9 15 +6
Lines 1964 3434 +1470
=========================================
Hits 127 127
- Misses 1837 3307 +1470
Impacted Files | Coverage Δ | |
---|---|---|
rfpy/scripts/rfpy_calc.py | 0.00% <0.00%> (ø) |
|
rfpy/scripts/rfpy_ccp.py | 0.00% <0.00%> (ø) |
|
rfpy/scripts/rfpy_harmonics.py | 0.00% <0.00%> (ø) |
|
rfpy/scripts/rfpy_hk.py | 0.00% <0.00%> (ø) |
|
rfpy/scripts/rfpy_plot.py | 0.00% <0.00%> (ø) |
|
rfpy/scripts/rfpy_recalc.py | 0.00% <0.00%> (ø) |
|
rfpy/__init__.py | 100.00% <100.00%> (ø) |
|
... and 3 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 727521f...d6814d0. Read the comment docs.
This PR is a re-factoring of the scripts to make them executable on the command line and packages them as
entry_points
instead ofscripts
. This fix is needed for Windows OS where the shebang is incorrectly interpreted and the scripts fail to run. It also forces removal of all reference to.py
when calling the scripts, and therefore requires a new subversion (0.1.1). All documentation has been updated to reflect this.