Skip to content
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

Unnecessary Action Runs on PRs #221

Open
davidgamero opened this issue Jul 5, 2023 · 0 comments
Open

Unnecessary Action Runs on PRs #221

davidgamero opened this issue Jul 5, 2023 · 0 comments

Comments

@davidgamero
Copy link
Collaborator

integration tests run on all PRs, even ones where only the README is changed.

Ref @hsubramanianaks 's comment on #220

We currently have several actions whose success is required in order to allow a PR to merge, so if we wanted to allow PRs for the README to merge without having to run all the tests, we may have to change the Required Checks which I'm not sure we can do per-file since last time I updated them it was a per-repository setting

Custom logic in the checks seems limited since you can't depend on other checks without some very custom tooling to probe the actions api, but if it's possible we should definitely do this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant