-
Notifications
You must be signed in to change notification settings - Fork 727
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
Bug fixes in v1.4 release #319
Conversation
Some of the version changes you sent werent consistent @apeltzer . Ive bumped everything to |
Some tests are failing? 😕 |
Ah, its because we bumped versions and tests cant find the appropriate container... |
Why are we still using |
Ah, GitHub actions files need to be updated too. Will need to add this into rnaseq/.github/workflows/ci.yml Line 10 in 55eee0c
|
We bumped it everywhere but in the GitHub Actions tests ... (which are not auto-bumped atm). |
Open an issue for that nf-core/tools#429 |
Ok. Tests are still failing because GitHub cant find the docker container for |
Looking much better. Do your thing with the patch release when youre happy 👍 |
Many thanks to contributing to nf-core/rnaseq!
To ensure that your build passes, please make sure your pull request is to the
dev
branch rather than tomaster
. Thank you!Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).
PR checklist
dev
rather thanmaster
nextflow run . -profile test,docker
).nf-core lint .
).docs
is updatedCHANGELOG.md
is updatedREADME.md
is updatedLearn more about contributing: https://github.com/nf-core/rnaseq/tree/master/.github/CONTRIBUTING.md
1.5dev