Skip to content

Add about the need to bidsignore dandiset.yaml#215

Merged
kabilar merged 3 commits intomasterfrom
enh-bids
Feb 2, 2026
Merged

Add about the need to bidsignore dandiset.yaml#215
kabilar merged 3 commits intomasterfrom
enh-bids

Conversation

@yarikoptic
Copy link
Member

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the BIDS validation documentation by adding important guidance about excluding DANDI-specific files from BIDS validation and expanding validation tool recommendations.

Key changes:

  • Adds instruction to use .bidsignore file for dandiset.yaml to prevent BIDS validation errors
  • Expands validation recommendations to include both BIDS validator and dandi validate command
  • Notes that dandi upload also performs validation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Kabilar Gunalan <kabi@mit.edu>
@kabilar kabilar merged commit fda298a into master Feb 2, 2026
0 of 3 checks 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