-
Notifications
You must be signed in to change notification settings - Fork 214
Add GHA to update template nf-test snapshots #3723
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
@nf-core-bot update textual snapshots |
@nf-core-bot update template snapshots |
@nf-core-bot update textual snapshots |
@nf-core-bot update textual snapshots |
Will need to merge this PR in order to test it |
name: Retrieve all template features | ||
# Only run if comment is on a PR with the main repo, and if it contains the magic keywords | ||
if: > | ||
contains(github.event.comment.html_url, '/pull/') && |
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.
do we need this?
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.
I think so? Otherwise, it would try to run on comments to issues too
needs: [prepare-matrix] | ||
# Only run if comment is on a PR with the main repo, and if it contains the magic keywords | ||
if: > | ||
contains(github.event.comment.html_url, '/pull/') && |
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.
same here
I am merging this PR to test it |
I don't think this is working #3766 (comment) @mirpedrol |
This GHA add an nf-core-bot option to update the template nf-test snapshots used in the tools CI.
The command is
@nf-core-bot update template snapshots
Close #3572
Website docs: nf-core/website#3478