Commit-queue should automatically handle PRs with merge commits from the target branch
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- git, javascript
Piste de recherche
Commencez par localiser le traitement de la commit queue qui invoque cherry-pick, puis reproduisez l’échec signalé avec une pull request contenant un merge depuis main. C’est terminé lorsque la commit queue traite automatiquement ces pull requests sans nécessiter de landing manuel ; l’issue ne nomme aucun fichier ni test à exécuter.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- JavaScript
- Étoiles
- 313
- Forks
- 137
- Merge moyen
- 2 j 8 h
- PR mergées (30 j)
- 24
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de nodejs/node-core-utils
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
nodejs/node-core-utils#664 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
nodejs/node-core-utils#1164 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 68/100
nodejs/node-core-utils#1160 · 3 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
nodejs/node-core-utils#1151 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
nodejs/node-core-utils#1063 ·
Toutes les issues de nodejs/node-core-utils
Issues similaires
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
avniproject/avni-client#2135 ·
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
babalae/bettergi-scripts-list#3674 ·
-
A-Release-Notes C-Editing D-Modest S-Ready-For-Implementation
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
bevyengine/bevy-website#2595 ·
-
ecosystem wording
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
matrix-org/matrix.org#3649 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
vadimdemedes/ink#1029 ·