-
Notifications
You must be signed in to change notification settings - Fork 11
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
Change code coverage trigger #178
Conversation
Backend code coverage report for PR #178 |
Frontend code coverage report for PR #178 |
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.
Perfect, thanks for catching 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.
🎉
Going to go ahead and merge to see how it works out! |
Motivation:
pull_request
is actually describing the target branch. So the check for the name of the PR branch is handled inif: !startsWith(github.head_ref, 'dependabot')
See the doc page here:
https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows
Description:This change makes it so that any PR branch which starts with
dependabot
skips the Code Coverage workflow.Part of grafana/code-coverage#17