-
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
[MAINT] remove code climate and update badges #2190
Conversation
Remi-Gau
commented
Nov 7, 2024
•
edited
Loading
edited
- remove code climate config
- update badges in README
- update links to legacy validator in README
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2190 +/- ##
=======================================
Coverage 85.70% 85.70%
=======================================
Files 91 91
Lines 3784 3784
Branches 1221 1221
=======================================
Hits 3243 3243
Misses 455 455
Partials 86 86 ☔ View full report in Codecov by Sentry. |
bids-validator/README.md
Outdated
@@ -335,6 +350,9 @@ see available options. | |||
|
|||
## Docker image | |||
|
|||
<!-- TODO update version --> | |||
**:rotating_light: docker images of the bids-validator used the legacy validator until version 1.X.Y** |
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.
not sure which version of the docker image contains the new validator.
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.
I don't think any do.
bids-validator/README.md
Outdated
<!-- TODO update version --> | ||
**:rotating_light: at this time (november 2024) docker images of the bids-validator still use the legacy validator. But this should soon change.** | ||
|
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.
changed the wording to let users know that they may still be using the legacy validator if using the docker image