Conversation

TheChymera

Codespell currently fails due to false positive detection of “inports”, which are input ports and not a misspelling of imports.
Also makes the code more clear I think.

@TheChymera

Not sure the same sort of fix is good for “expad” (similarly ex-padding), because that's a verbatim parameter for AFNI... probably not a good idea to introduce an underscore there.

@codecovCodecov

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (5ef9fe4) 63.15% compared to head (71e8795) 63.15%.

Files%Lines
nipype/pipeline/engine/utils.py78.94%2 Missing and 2 partials⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           master    #3627    +/-   ##
========================================
  Coverage   63.15%   63.15%            
========================================
  Files         308      308            
  Lines       40825    40825            
  Branches     5656     5500   -156     
========================================
  Hits        25781    25781            
  Misses      14031    14031            
  Partials     1013     1013            

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

@effigieseffigies merged commit 42302ed into nipy:master Feb 6, 2024
@TheChymeraTheChymera deleted the codespell branch February 6, 2024 19:55
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.