-
Notifications
You must be signed in to change notification settings - Fork 13
Provide option to run locally installed bids-validator #380
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
Conversation
tien-tong
left a comment
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.
add docstrings to the functions you're editing, if they don't have it already, or only had partial infor
tien-tong
left a comment
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.
please fix lint issues, you can run black on the script and push your changes until it passes lint test
|
Done! |
|
@gabridele can you also add instruction on how to download the new schema validation via deno to the documentation? |
|
@tientong98 will do, as soon as I'm done with another issue |
Closes #376
Changes proposed in this pull request
--local-validatorargument to thecubids validatecommand, it runs the locally installed bids-validator. For use cases where HPCs don't have internet accessbuild_validator_callhas now if statement to accommodate choice of local validatorDocumentation that should be reviewed
cubids validateso far)