-
Notifications
You must be signed in to change notification settings - Fork 630
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
Fix failing CI tests #4861
Fix failing CI tests #4861
Conversation
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
Signed-off-by: Ben Sherman <bentshermann@gmail.com>
✅ Deploy Preview for nextflow-docs-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@pditommaso don't know if you did something with the github secrets but unit tests are passing here 👍 |
LGTM👍 |
This reverts commit 1c0e648.
Signed-off-by: Ben Sherman <bentshermann@gmail.com> Signed-off-by: Niklas Schandry <niklas@bio.lmu.de>
This reverts commit 1c0e648. Signed-off-by: Niklas Schandry <niklas@bio.lmu.de>
Signed-off-by: Ben Sherman <bentshermann@gmail.com> Signed-off-by: Niklas Schandry <niklas@bio.lmu.de>
This reverts commit 1c0e648. Signed-off-by: Niklas Schandry <niklas@bio.lmu.de>
This reverts commit 1c0e648. Signed-off-by: Niklas Schandry <niklas@bio.lmu.de>
@pditommaso why was this PR reverted? even if it wasn't related to the google batch CI tests, they are still good changes. the docs test actually will fail sometimes without this change |
Becasue |
Then keep the docs snippet? |
Fair enough 6499649 |
see discussion in #4712