-
Notifications
You must be signed in to change notification settings - Fork 5
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
[REF] Rename Neurobagel-created session to ses-unnamed
#358
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #358 +/- ##
=======================================
Coverage 98.37% 98.37%
=======================================
Files 16 16
Lines 983 983
=======================================
Hits 967 967
Misses 16 16 ☔ View full report in Codecov by Sentry. |
Pull Request Test Coverage Report for Build 11263495274Details
💛 - Coveralls |
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.
Looks good @alyssadai, thanks for this! I cannot believe how long that issue has been open 👀
I left two 🍒 suggestions for clarity. Take a look, and then 🧑🍳
Co-authored-by: Sebastian Urchs <[email protected]>
🚀 PR was released in |
Checklist
This section is for the PR reviewer
[ENH]
,[FIX]
,[REF]
,[TST]
,[CI]
,[MNT]
,[INF]
,[MODEL]
,[DOC]
) (see our Contributing Guidelines for more info)skip-release
(to be applied by maintainers only)Closes #XXXX
For new features:
For bug fixes: