-
Notifications
You must be signed in to change notification settings - Fork 165
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
[MISC] Reorder appendices #1256
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1256 +/- ##
=======================================
Coverage 88.53% 88.53%
=======================================
Files 6 6
Lines 1055 1055
=======================================
Hits 934 934
Misses 121 121
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Verifying https://bids-specification--1256.org.readthedocs.build/en/1256/99-appendices/14-glossary.html redirects correctly. |
Co-authored-by: Chris Markiewicz <effigies@gmail.com>
src/04-modality-specific-files/01-magnetic-resonance-imaging-data.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Chris Markiewicz <effigies@gmail.com>
Co-authored-by: Chris Markiewicz <effigies@gmail.com>
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.
if we merge this we need to push adjustments to the NIRS pr
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.
Had a quick look and nothing stands out as out of order.
src/04-modality-specific-files/01-magnetic-resonance-imaging-data.md
Outdated
Show resolved
Hide resolved
This now has three approvals. I'm thinking this change is big enough that we should wait the 5 days. What do you all think? |
thanks @tsalo |
Closes #1204.
Changes proposed: