-
Notifications
You must be signed in to change notification settings - Fork 111
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
feat: Validate CITATION.cff #2116
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2116 +/- ##
==========================================
+ Coverage 85.72% 87.58% +1.86%
==========================================
Files 91 134 +43
Lines 3782 7017 +3235
Branches 1220 1660 +440
==========================================
+ Hits 3242 6146 +2904
- Misses 454 776 +322
- Partials 86 95 +9 ☔ View full report in Codecov by Sentry. |
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.
LGTM. CI failures should be resolved in #2117.
…tation.cff project.
Co-authored-by: Chris Markiewicz <[email protected]>
Co-authored-by: Chris Markiewicz <[email protected]>
8689e55
to
0c1b20a
Compare
Dataset level check to validate citation.cff. Imports citation.cff json schema from @bids/schema package, convert citation.cff to json, and call ajv on it with stock ajv-formats.
Closes #1959.