Conversation

DimitriPapadopoulos

No description provided.

@DimitriPapadopoulos
STY: Apply ruff rule RUF017
e01173d
RUF017 Avoid quadratic list summation
@DimitriPapadopoulos
STY: Apply ruff rule RUF021
1be9a67
RUF021 Parenthesize `a and b` expressions when chaining `and` and `or` together,
       to make the precedence clear
@codecovCodecov

Codecov Report

Attention: coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 70.89%. Comparing base (83e3903) to head (5ce526f).
Report is 112 commits behind head on master.

Files with missing lines%Lines
nipype/interfaces/fsl/utils.py0.00%1 Missing⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3688      +/-   ##
==========================================
- Coverage   70.89%   70.89%   -0.01%     
==========================================
  Files        1277     1277              
  Lines       59212    59216       +4     
  Branches     9799     9803       +4     
==========================================
  Hits        41980    41980              
- Misses      16066    16069       +3     
- Partials     1166     1167       +1

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

effigies
effigies reviewed Oct 6, 2024
View reviewed changes
@DimitriPapadopoulos @effigies
STY: Further simplification
5ce526f
Co-authored-by: Chris Markiewicz <[email protected]>
@effigieseffigies merged commit d3cfb41 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

@effigies effigies effigies approved these changes

No one assigned
None yet
None yet
No milestone

Successfully merging this pull request may close these issues.