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

[PR #10315/dc65bb6a backport][8.2.x] Introduce a gate/check GHA job #12484

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jun 19, 2024

This is a backport of PR #10315 as merged into main (dc65bb6).

This adds a GHA job that reliably determines if all the required
dependencies have succeeded or not.

It also allows to reduce the list of required branch protection CI
statuses to just one — check. This reduces the maintenance burden
by a lot and have been battle-tested across a small bunch of projects
in its action form and in-house implementations of other people.

I was curious about the spread of use. And I've just discovered that
it is now in use in aiohttp (and other aio-libs projects), CherryPy,
some of the Ansible repositories, all of the jaraco's projects (like
setuptools, importlib_metadata), some PyCQA, PyCA and pytest
projects, a few AWS Labs projects. Admittedly, I maintain a few of
these but it seems to address some of the pain folks have:
jaraco/skeleton#55 (comment).

The story behind this is explained in more detail at
https://github.com/marketplace/actions/alls-green#why.

Introduce a gate/check GHA job

(cherry picked from commit dc65bb6)
@webknjaz
Copy link
Member

@RonnyPfannschmidt approve & auto-merge ^

@RonnyPfannschmidt RonnyPfannschmidt merged commit 4716ba5 into 8.2.x Jun 19, 2024
26 of 28 checks passed
@RonnyPfannschmidt RonnyPfannschmidt deleted the patchback/backports/8.2.x/dc65bb6a6699bdeb5ee7523245db1ee724670e29/pr-10315 branch June 19, 2024 09:18
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

Successfully merging this pull request may close these issues.

2 participants