Skip to content

docs: describe prerequisite files with many links#45

Merged
yarikoptic merged 2 commits into
bids-standard:masterfrom
WillForan:readme-updates
Jul 13, 2025
Merged

docs: describe prerequisite files with many links#45
yarikoptic merged 2 commits into
bids-standard:masterfrom
WillForan:readme-updates

Conversation

@WillForan

Copy link
Copy Markdown
Contributor

Descriptions, examples, and references for participants.tsv and scans.tsv.
Update note: ExperimentID is read from json for bold.

Link to NDA image03 and GUID Tools, nda-tools, web uploader, bids-validator, and BIDS spec for required but BIDS-optional files.

Descriptions, examples, and references for participants.tsv and scans.tsv.
Update note: ExperimentID is read from json for bold.

Link to NDA image03 and GUID Tools, nda-tools, web uploader, bids-validator, and BIDS spec for required but BIDS-optional files.

@yarikoptic yarikoptic left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor formatting

Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Co-authored-by: Yaroslav Halchenko <debian@onerussian.com>
@WillForan

Copy link
Copy Markdown
Contributor Author

Thanks! Updated with all the changes.

Aside: I clicked commit changes on each suggestion in the webUI and then rebased locally. Any idea if there is more ergonomic way to accept all and collapse? Did I lose any important history or authorship doing that?

@yarikoptic

Copy link
Copy Markdown
Collaborator

Any idea if there is more ergonomic way to accept all and collapse?

you can stage multiple recommended changes and commit in a single commit while in "Files changed" tab

Did I lose any important history or authorship doing that?

I do not think so, looks good to me.

@yarikoptic
yarikoptic merged commit 0c7efb4 into bids-standard:master Jul 13, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants