Assigning PRs to merger doesn't work when they're auto-merged

Offen

#2,137 erstellt am 13.2.2024 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: github-actions, python
Bereich: ci-cd, devops

Rechercherichtung

Beginnen Sie mit .github/update-release-branch.py in den Zeilen 163-168 und prüfen Sie das Beispiel-PR #2110, um zu verstehen, wie die Zuweisung für automatisch gemergte Pull Requests abgeleitet wird. Bestätigen Sie das vorgesehene Assignee-Verhalten für diesen Fall; die Aufgabe ist abgeschlossen, wenn automatisch gemergte PRs gemäß dieser Regel zugewiesen werden, anstatt fälschlicherweise demjenigen zugewiesen zu werden, der den Merge durchgeführt hat.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

A bunch of [@jsoref's] PRs are assigned to the people who merged them. It feels like the automation hasn't contemplated that PRs would be written by people who can't/won't be the people who will click the "merge" button.

This is intentional:
https://github.com/github/codeql-action/blob/bc64d12bb9f349435efba65d373bac054665b85f/.github/update-release-branch.py#L163-L168

More interesting is that this approach seems to break when enabling auto-merge as can be seen here:
(merged by @angelapwen but PR is from @jsoref)