pr-trigger.yml: post link to triggered workflows - #388
Conversation
|
Triggered workflows for this PR:
|
|
@luhenry this is a follow-up to my trigger rework - it posts a link to the triggered action. I thought the I'll see if I can make it cleaner. |
A per-package pull_request: paths trigger means any PR touching several workflow files fires that many separate (mostly irrelevant) runs, flooding the Actions list. workflow_dispatch is now the only way any of these run; pr-trigger.yml's Checks-API relay gives the same visible, per-directive status without the trigger sprawl. Co-authored-by: Claude Opus 5 <noreply@anthropic.com> Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
Trigger: directives dispatch via workflow_dispatch, which has no PR check of its own. Post one GitHub Check Run per directive instead of a relay job or a PR comment: a properly named, real check (build-<pkg>.yml @ <version>) whose "Details" link goes straight to the dispatched run, with real pending/success/failure/cancelled status mirrored via `gh run watch`. Co-authored-by: Claude Opus 5 <noreply@anthropic.com> Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
Signed-off-by: Trevor Gamblin <tgamblin@baylibre.com>
13c06a0 to
dd780d5
Compare
|
I think that this is much better. If you click through the links in the web UI, you can see all of the builds running for each package and version: https://github.com/riseproject-dev/python-wheels/runs/97938652031 |
|
Dropping this until I can figure out the right logic for all trigger cases. |
We use a
Trigger: <package>:<version>pattern in PRs to tell our workflows which versions to build for a given change, but this doesn't automatically show the actual workflows which have been triggered, and it's confusing regarding whether the default version should have aTriggerdirective or not. We also don't necessarily want changes to workflows submitted with a PR to automatically trigger those workflows, since they can run for a long time and many changes may be simple maintenance cleanups or improvements.Now, all PRs require the
Triggerdirective even for the default PR, and versions other than what we need should still trigger and appear in the PR UI.Trigger: pytokens:0.4.1
Trigger: numpy:2.5.0