| 13 Oct 2023 |
infinisil | But it did use an updated base branch, just due to how it fetches the merge ref, which github always updates automatically:
- Before base update: https://github.com/tweag/nixpkgs/actions/runs/6500830578/attempts/1?pr=74#summary-17657001189
- After base update: https://github.com/tweag/nixpkgs/actions/runs/6500830578?pr=74#summary-17657034351
(note how the stated base branch commit changed)
| 14:55:13 |
Robert Hensing (roberth) | unfortunate :/ | 15:59:33 |
infinisil | Hmm but even then it's not perfect, PR's before the introduction of the pkgs/by-name check wouldn't be failing :/ | 16:42:26 |
infinisil | Alternative is to somehow force all PR's to be rebased | 16:42:57 |
infinisil | Or even automatically rebase? No that would mess with GPG signatures | 16:43:36 |
infinisil | Merging master automatically into each PR? No that would be a super messy history | 16:43:50 |
infinisil | Maybe it's possible to just kind of associate a new failing check run with PR's that should be rebased | 16:44:47 |
infinisil | Let me try experimenting with that | 16:45:18 |
infinisil | Aha! This looks pretty good https://docs.github.com/en/rest/commits/statuses?apiVersion=2022-11-28 | 16:49:02 |