Azure / draft

A day 0 tool for getting your app on k8s fast
MIT License
515 stars 61 forks source link

Unnecessary Action Runs on PRs #221

Open davidgamero opened 1 year ago

davidgamero commented 1 year ago

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!