New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[MAINT] Remove unnecessary lines describe in test_clean_psc
#4208
Conversation
…es a useless line nilearn#4175. This caused a third line to be unnecessary so I removed it also.
👋 @paul-reiners Thanks for creating a PR! Until this PR is ready for review, you can include the [WIP] tag in its title, or leave it as a github draft. Please make sure it is compliant with our contributing guidelines. In particular, be sure it checks the boxes listed below.
For new features:
For bug fixes:
We will review it as quick as possible, feel free to ping us with questions if needed. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4208 +/- ##
=======================================
Coverage 92.09% 92.09%
=======================================
Files 144 144
Lines 16366 16366
Branches 3426 3426
=======================================
Hits 15073 15073
Misses 754 754
Partials 539 539
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
test_clean_psc
test_clean_psc
test_clean_psc
Co-authored-by: Yasmin <63292494+ymzayek@users.noreply.github.com>
Believe it's ready to go now. Thanks for all your help so far. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thx.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks
thanks @paul-reiners |
This caused a third line to be unnecessary so I removed it also.
test_clean_psc
includes a useless line #4175Changes proposed in this pull request: