Dry runs of release-pr and prerelease edit the checkout

Aperta Adatta ai principianti
  • Qualcosa non va?
#134 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
62/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
bun, git, typescript
Ambito
devtools, release

Direzione di ricerca

Inizia dai controlli di dry run in src/release-pr/index.ts intorno alle righe 162–166 e in src/prerelease/index.ts intorno alla riga 97; segui il checkout, gli aggiornamenti di versione e gli script di release eseguiti prima. Decidi come i dry run dovrebbero evitare di modificare il checkout, oppure se il README dovrebbe indirizzare gli utenti verso un clone usa e getta. Il lavoro è concluso quando un dry run locale lascia invariato il checkout oppure il README documenta chiaramente la soluzione alternativa.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug
Issue description

The README recommends trying an action locally as a dry run (INPUT_DRY_RUN=true bun dist/release-pr/index.js). Before it checks dry, release-pr runs git checkout -B <release branch>, rewrites every selected package.json version and runs each package's release script, which writes its CHANGELOG.md (src/release-pr/index.ts#L162-L166). prerelease also calls setVersion before its dry check (src/prerelease/index.ts#L97).

So a local dry run leaves the developer on a new branch with modified files. Traced, not run (to keep the checkout clean). In CI it doesn't matter.

Expected: a dry run doesn't touch the checkout, or the README says to run it in a throwaway clone.

Found by a read-only audit of main at 435a16b (first done at 046494f; no src/ file changed in between), report audits/actions-2026-11-11.md in the workspace repo.

Code sample
await checkoutBranch(branch);
await applyVersions(group, plans);

let url = "";
if (dry) {
  info(`[DRY] Would push "${branch}" and open the PR "${message}"`);
Versions
  • @nanoforge-dev/actions 3.3.0 (main at 435a16b)
  • Bun 1.3.14
Issue priority

Low (slightly annoying)

I have tested this issue on a development release

435a16b

Lingua principale
TypeScript
Stelle
0
Fork
0
Merge medio
15h 51m
PR unite (30g)
21

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di NanoForge-dev/actions

Tutte le issue di NanoForge-dev/actions

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.