nodejs / nodejs/node-core-utils
Commit-queue should automatically handle PRs with merge commits from the target branch
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 314
- Forks
- 137
- Ø Merge
- 2 T. 8 Std.
- Gemergte PRs (30 T.)
- 24
Beschreibung
Some contributors merge into their PR branch from main rather than rebasing. We might not recommend this, but it shouldn't be a major issue. Unfortunately it seems that the commit-queue label can't handle this, and fails with:
error: commit ... is a merge but no -m option was given.
fatal: cherry-pick failed
Here's a example I just ran into: https://github.com/nodejs/node/pull/59375#issuecomment-3178324429. More examples here.
AFAICT this then always requires manually landing the PR, but it feels like we should be able to handle these automatically without much trouble.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Verarbeitung der commit queue zu lokalisieren, die cherry-pick aufruft, und reproduziere dann den gemeldeten Fehler mit einem pull request, der einen merge von main enthält. Als erledigt gilt die Aufgabe, wenn die commit queue solche pull requests automatisch verarbeitet, ohne dass ein manuelles Landing erforderlich ist; im issue werden keine Dateien oder auszuführenden Tests genannt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- git, javascript
- Bereich
- cli, tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100