-
Notifications
You must be signed in to change notification settings - Fork 378
PR checks: support collections of workflows #3030
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for collections of workflows in the PR checks system. It allows grouping related workflow checks (like Go-related checks) into collections that can be executed together.
- Adds collection functionality to the PR checks sync script
- Introduces workflow inputs for parameterizing check execution
- Creates a consolidated "go" collection containing all Go-related checks
Reviewed Changes
Copilot reviewed 64 out of 64 changed files in this pull request and generated 1 comment.
File | Description |
---|---|
pr-checks/sync.py | Main implementation adding collection support and workflow input handling |
pr-checks/checks/go-*.yml | Go check specifications updated with collection membership and inputs |
.github/workflows/__go.yml | New generated collection workflow for running all Go checks together |
.github/workflows/__*.yml | Generated workflow files updated with workflow_call and workflow_dispatch inputs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! I like this. There is a little bit of awkwardness in the "go-version" input not being used in any of the steps, but it's not blocking.
c3ee05b
to
20c329c
Compare
I manually dispatched a run of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. Thanks for the extra cleanup!
This is a sort-of experimental idea that I had in the context of #3028 where we bump the Go version in the workflows to a release candidate for testing purposes, but such PRs don't have to be merged anymore because the existing constraint of
>= 1.22
would include allow new version once it is fully released.This PR updates the
sync.py
script to allow "collections" of PR checks that can be triggered manually, with specific inputs. So instead of creating a PR that bumps the Go version for testing purposes only, we could instead just dispatch the collection workflow with the desired Go version.Risk assessment
For internal use only. Please select the risk level of this change:
Merge / deployment checklist