-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add CI to contributing #703
Conversation
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.
Placement here looks fine. I'd just maybe rephrase about what is required, as spelling is not.
- [Check Nextflow config](.github/workflows/nextflow-config-check.yaml): This workflow ensures that there are no syntax errors in the Nextflow configuration files | ||
|
||
There is also one additional `pre-commit ci` workflow which runs all [pre-commit hooks as described in this section](#pre-commit-hooks), except for the spell check pre-commit hook. | ||
Although highly recommended, it is not required that this workflow passes before pull requests can be merged. |
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.
Spell check is not required to pass either, the way this repo is currently set up.
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.
Ah, didn't know that! Also looks like the precommit ci only runs on development
(if i'm reading the yaml syntax correctly), so will change that too.
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.
No, it runs everywhere. That is where it files its PRs to (when updating hook versions)
Co-authored-by: Joshua Shapiro <josh.shapiro@ccdatalab.org>
Closes #695
Here I've added a small section about CI into contributing. I nested this inside the PRs header and mentioned both required workflows and the pre-commit ci workflow (do I still have it correct that passing is not mandatory? I don't have access to Settings here so couldn't tell for sure).
Let me know if this is the right place in
CONTRIBUTING.md
to have added this, as well as any details to be expanded upon or reduced.