Conversation

DimitriPapadopoulos

Summary

Apply flake8-pie (PIE) rules, except:

@codecovCodecov

Codecov Report

Attention: coverage is 20.00000% with 12 lines in your changes missing coverage. Please review.

Project coverage is 70.84%. Comparing base (e03ab6f) to head (3980037).
Report is 176 commits behind head on master.

Files with missing lines%Lines
nipype/interfaces/cmtk/cmtk.py0.00%2 Missing⚠️
nipype/interfaces/dcm2nii.py0.00%2 Missing⚠️
nipype/algorithms/misc.py0.00%1 Missing⚠️
nipype/interfaces/cmtk/nx.py0.00%1 Missing⚠️
nipype/interfaces/dipy/simulate.py0.00%1 Missing⚠️
nipype/interfaces/freesurfer/preprocess.py0.00%1 Missing⚠️
nipype/interfaces/freesurfer/utils.py0.00%0 Missing and 1 partial⚠️
...ne/plugins/tests/test_legacymultiproc_nondaemon.py0.00%1 Missing⚠️
nipype/utils/filemanip.py0.00%1 Missing⚠️
nipype/utils/nipype2boutiques.py0.00%1 Missing⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3678   +/-   ##
=======================================
  Coverage   70.83%   70.84%           
=======================================
  Files        1276     1276           
  Lines       59320    59313    -7     
  Branches     9826     9826           
=======================================
- Hits        42019    42018    -1     
+ Misses      16125    16119    -6     
  Partials     1176     1176           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

PIE790 Unnecessary `pass` statement
PIE804 Unnecessary `dict` kwargs
PIE808 Unnecessary `start` argument in `range`
PIE810 Call `endswith` once with a `tuple`
PIE810 Call `endswith` once with a `tuple`
@effigieseffigies merged commit 72a944e into nipy:master Oct 6, 2024
19 checks passed
Sign up for free to join this conversation on . Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.