-
Notifications
You must be signed in to change notification settings - Fork 8
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
proposal: ignore push events for dependabot branches #56
Comments
Will take a look. Less CI runs would mean less resources/bandwidth wasted and less time waiting. |
Exactly. When we e.g. activated license-check it always run twice, clogging the github runners. Thank you. :) |
something like the following would solve this, not just for Dependabot PRs but for all PRs: on:
push:
branches:
- main
- master
- next
paths-ignore:
- 'docs/**'
- '*.md'
pull_request:
paths-ignore:
- 'docs/**'
- '*.md' Your recent PR to fastify-helmet ran it twice as well: fastify/fastify-helmet#195 |
@Fdawgs are you using https://github.com/Eomm/massive-wax? |
No, just using a horrible shell script 😂 |
PR made to all Fastify repos to add this. |
Prerequisites
Issue
When dependabot opens a PR to update a dependency, it runs the whole ci pipeline twice. Once by push and once by pull request event. We could actually save one ci run when we would ignore the dependabot branch in push events.
We could actually configure, that only push events into master, main and next result in a pipeline run.
The text was updated successfully, but these errors were encountered: