-
Notifications
You must be signed in to change notification settings - Fork 688
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
Use new nf-test features #6286
base: master
Are you sure you want to change the base?
Use new nf-test features #6286
Conversation
edmundmiller
commented
Aug 23, 2024
- ci: Attempt to split everything out
- ci: Add changed since, sharding, and ci
- ci: Add filter to try to get jobs split up
- ci: Switch to only-changed
- ci: See if follow-dependencies works without "related-tests"
489e5fb
to
26653c8
Compare
That would close #391 as well, and might encourage people to make smaller PRs if we're only giving them 9 CI runners per PR anyway. |
Don't hate me Adam
Idk what we're going to do about these...
6030ae8
to
5594071
Compare
Current things blocking this is nf-core/tools#3140 or we can give up and keep the paths-filter step for that |
c4f6257
to
9cd5419
Compare
9cd5419
to
8748f04
Compare
Removed the need to use nf-core lint with pre-commit, as that wasn't truely blocking this from getting merged. We can make a follow up issue to run every linting step through pre-commit. |
Looking good, there's still some leftover comments in the nf-test yml, but I like it |