RFR: Mark integrated Pull Requests as properly merged in GitHub repositories
Julian Waters
jwaters at openjdk.org
Thu Nov 3 05:24:05 UTC 2022
Skara currently closes integrated Pull Requests, since the actual integration is done internally and then pushed to the repository separately. This makes every integrated request always look like it was closed to an outside observer, forcing the use of a special integrated flag to distinguish between integrated and closed, or rejected and closed. On top of that, it just looks awful in the interface all around. In the [willful absence of a reply from the GitHub team to allow marked a pull as merged through a flag](https://github.com/orgs/community/discussions/12437), and GitLab being unlikely to make a similar change either, a workaround with minimal impact can be used instead: Every pull has a corresponding pre-integration branch created at the time it was made, and this special branch is marked for deletion when the pull is integrated. Following the principle of waste not, want not, a change that is currently only implemented for GitHub repositories instead resets the pre-integratio
n branch to the source ref of the pull request, then merges the request into it's own pre-integration branch before the actual deletion occurs, leaving minimal impact on most of Skara's infrastructure and with virtually no effect on already existing repositories.
- [x] Properly handle all dependencies of the pre-integration branch before it's modified irreversibly
-------------
Commit messages:
- Squash for neatness
Changes: https://git.openjdk.org/skara/pull/1409/files
Webrev: https://webrevs.openjdk.org/?repo=skara&pr=1409&range=00
Stats: 101 lines in 7 files changed: 51 ins; 19 del; 31 mod
Patch: https://git.openjdk.org/skara/pull/1409.diff
Fetch: git fetch https://git.openjdk.org/skara pull/1409/head:pull/1409
PR: https://git.openjdk.org/skara/pull/1409
More information about the skara-dev
mailing list