nodejs / nodejs/node-core-utils
`git node release --promote` doesn't support worktrees
Une pull request liée a déjà été fusionnée.
- #907 par @aduh95 — fusionnée
- Langage dominant
- JavaScript
- Étoiles
- 314
- Forks
- 137
- Merge moyen
- 2 j 8 h
- PR mergées (30 j)
- 24
Description
On my computer, I have several git worktrees (one for main, one for each release line, etc).
The promotion fails on the "cherry-pick to main" step, because I'm running it from the v23.x worktree and main is checked out in another one:
✔ Cherry-pick release commit to the default branch? Yes
fatal: 'main' is already used by worktree at '/Users/mzasso/git/nodejs/node'
Error: git checkout,main failed: 128
at ChildProcess.<anonymous> (file:///Users/mzasso/.volta/tools/image/packages/@node-core/utils/lib/node_modules/@node-core/utils/lib/run.js:45:21)
at ChildProcess.emit (node:events:507:28)
at maybeClose (node:internal/child_process:1101:16)
at ChildProcess._handle.onexit (node:internal/child_process:305:5)
at Process.callbackTrampoline (node:internal/async_hooks:130:17) {
code: 128,
messageOnly: true
}
I don't know if that can be fixed.
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.
Piste de recherche
Commencez par exécuter git node release --promote depuis un release-line worktree, tandis que main est checked out dans un autre worktree. Lisez l’étape de cherry-pick vers main et son échec avec git checkout,main, puis vérifiez que la promotion réussit sans entrer en conflit avec le worktree existant.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- git, javascript, nodejs
- Domaine
- cli, release, tooling
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 25/100