-
Notifications
You must be signed in to change notification settings - Fork 7
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
[ENH] Clearer DOCs for session and participant IDs #226
Conversation
✅ Deploy Preview for neurobagel-documentation ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Also some overlength line breaks
- Also include a clarification in the section on multiple participants or sessions
We really should get codespell to run locally
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.
Hey @surchs, thanks for the PR!
A few suggestions below, incl. one about the overall docs structure - lmkwyt!
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.
Looking good!! One 🍒 below I just thought of, otherwise 🧑🍳
Co-authored-by: Alyssa Dai <[email protected]>
Changes proposed in this pull request:
Checklist
Please leave checkboxes empty for PR reviewers
[ENH]
,[FIX]
,[REF]
,[TST]
,[CI]
,[MNT]
,[INF]
) see our Contributing Guidelines for more info)Closes #XXXX