nodejs / nodejs/node-core-utils
Commit-queue should automatically handle PRs with merge commits from the target branch
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 314
- Forks
- 137
- Merge medio
- 2 d 8 h
- PR fusionados (30 d)
- 24
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.
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.
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.
Evaluación
- Stack tecnológico
- git, javascript
- Área
- cli, tooling
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100