Commit-queue should automatically handle PRs with merge commits from the target branch
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- git, javascript
Línea de trabajo
Empieza por localizar el procesamiento de commit queue que invoca cherry-pick y, después, reproduce el fallo indicado con un pull request que contenga un merge desde main. Se considera terminado cuando commit queue procesa automáticamente esos pull requests sin requerir un landing manual; el issue no indica ningún archivo ni ninguna prueba que ejecutar.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- JavaScript
- Estrellas
- 313
- Forks
- 137
- Merge medio
- 2 d 8 h
- PR fusionados (30 d)
- 24
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nodejs/node-core-utils
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
nodejs/node-core-utils#664 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 65/100
nodejs/node-core-utils#1164 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 68/100
nodejs/node-core-utils#1160 · 3 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
nodejs/node-core-utils#1151 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
nodejs/node-core-utils#1063 ·
Todos los issues de nodejs/node-core-utils
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
avniproject/avni-client#2135 ·
-
automated broken-link
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
agent/security hive/hosted-available-lke648397-260827-5n31 security
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
babalae/bettergi-scripts-list#3674 ·
-
A-Release-Notes C-Editing D-Modest S-Ready-For-Implementation
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
bevyengine/bevy-website#2595 ·