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
[DOC] Adding Himanshu to authors #4322
Conversation
👋 @man-shu 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. |
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.
make sure you also add yourself to the citation.cff file otherwise the sphinx reference for you (see below) won't be created.
`Himanshu Aggarwal`_
Turns out I was already in it! Co-authored a contribution a few years ago. |
For context: If it is modified, a CI workflow will run This script is also run before any build of the doc in CI. If the why or how is not clear, let me know so that we can improve the dev-facing doc. |
hahaha: silly me!!!! 🙈 OK this should be good to go then. |
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.
something must have gone wrong on the merge with upstream because some line changes should not appear here
assuming that the central of nilearn is a remote called upstream
on your local set up can you run
git fetch --all
git merge upstream/main
git push
Ok so now we only have the needed changes. I'll make sure to checkout to |
no worries at all, it still happens to me all the time!!! 😉 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4322 +/- ##
==========================================
+ Coverage 91.85% 92.05% +0.19%
==========================================
Files 144 143 -1
Lines 16419 16452 +33
Branches 3434 3451 +17
==========================================
+ Hits 15082 15145 +63
+ Misses 792 760 -32
- Partials 545 547 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Changes proposed in this pull request: