watch: a rebuild queued during a failed deploy is never deployed
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- node.js, typescript
Direzione di ricerca
Inizia da packages/cli/src/watch.ts e segui come triggerDeploy gestisce i percorsi di successo e di errore della promise. Riproduci il problema salvando una modifica durante un deploy in corso, poi assicurati che il deploy in coda venga eseguito dopo entrambi gli esiti; verifica che non sia bloccato dal blocco del deploy descritto in #47.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
In packages/cli/src/watch.ts, if a rebuild happens while a deploy is running, triggerDeploy sets deployQueued = true. The .then handler starts the queued deploy, but the .catch handler only resets isDeploying and never checks deployQueued.
So if the running deploy fails, a change saved during it is silently left undeployed until the next file change. That's often exactly the fix for the failure.
Repro
- In
notation watch, start a deploy that will fail. - While it's running, save a change.
- The failed deploy finishes, and the saved change is not deployed until another edit.
Fix
Drain the queue in both paths, e.g. move the deployQueued check into a .finally, or call the same "done" function from .then and .catch.
Note that while #47 is open, the queued deploy would then hang on the deployment hold, so the two are worth fixing together.
- Lingua principale
- TypeScript
- Stelle
- 54
- Fork
- 1
- Merge medio
- 1h 45m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di notationlabs/notation
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
notationlabs/notation#47 ·
-
state provider configAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
notationlabs/notation#34 ·
-
envs as configAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
notationlabs/notation#33 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
notationlabs/notation#16 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
notationlabs/notation#15 ·
Tutte le issue di notationlabs/notation
Issue simili
-
[Feature]: 建议导入源后,自动切换到新导入的源Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
lyswhut/lx-music-desktop#3026 ·
-
area: ci type: test
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
schlessera/brain-kit#2769 ·
I maintainer di solito rispondono entro 1 giorno
-
Read a vector index's quantization back on Neo4j 2026.10, which reports vector.quantization.typeAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
liberation-data/drivine#231 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
alveusgg/alveusgg#2502 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno