Skip to content

BIDS-2.0#1775

Draft
yarikoptic wants to merge 13 commits intomasterfrom
bids-2.0
Draft

BIDS-2.0#1775
yarikoptic wants to merge 13 commits intomasterfrom
bids-2.0

Conversation

@yarikoptic
Copy link
Copy Markdown
Collaborator

@yarikoptic yarikoptic commented Apr 12, 2024

References

Plan

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 12, 2024

Codecov Report

Attention: Patch coverage is 80.48780% with 16 lines in your changes are missing coverage. Please review.

Project coverage is 87.50%. Comparing base (f0bc977) to head (59fa77e).

Files Patch % Lines
tools/schemacode/bidsschematools/migrations.py 86.36% 9 Missing ⚠️
tools/schemacode/bidsschematools/__main__.py 0.00% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1775      +/-   ##
==========================================
- Coverage   87.92%   87.50%   -0.43%     
==========================================
  Files          16       17       +1     
  Lines        1375     1456      +81     
==========================================
+ Hits         1209     1274      +65     
- Misses        166      182      +16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

# Run only on a non-merge commit for the PR
if: >
github.repository == 'bids-standard/bids-specification' &&
github.event.pull_request.head.ref == 'bids-2.0' &&
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

might want to extend here to any branch with bids-2.0 prefix but without -patched suffix so we support rendering of diffs etc for feature branches against this branch, such as in

@yarikoptic yarikoptic changed the title WiP BIDS-2.0 BIDS-2.0 May 15, 2024
Copilot AI added a commit that referenced this pull request Dec 18, 2025
- Add workflow based on PR #1775 to test migrations on example datasets
- Uses bst migrate command to migrate datasets from bids-examples
- Runs validation before and after migration with deno validator
- Skips check-only migrations (inheritance overloading and TSV prefix checks)
- Adapted to use new 'bst migrate all' command instead of 'bst migrate-datasets'

Co-authored-by: yarikoptic <39889+yarikoptic@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant