Formatter drops a comment before the spread of a list pattern like `list{/* c */ ...r}`
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #9127 di @cknitt — integrata
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 56/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- ocaml
- Ambito
- compilers
Direzione di ricerca
Start by reading the formatter and pattern printing code for list patterns, then trace how comments attached to a spread are handled when list{...r} parses to r. Reproduce with the supplied bsc -format example and compare it with the expression case and the list{a, /* e */ ...r} case. Done means formatting preserves /* d */ in the pattern.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Formatting drops a comment written before the spread of a list pattern that has no other items.
let f = x =>
switch x {
| list{/* d */ ...r} => r
}
bsc -format (master at 3562ce6b80) prints:
let f = x =>
switch x {
| r => r
}
The comment /* d */ is gone. list{...r} parses to the pattern r itself, so the printer prints r, and the comment attached to the spread is never printed. The expression list{/* h */ ...xs} keeps its comment (/* h */ xs), and list{a, /* e */ ...r} keeps it too (with #9119).
Found while working on #9107; #9119 does not change this case.
🤖 Generated with Claude Code
- Lingua principale
- OCaml
- Stelle
- 7.5k
- Fork
- 484
- Merge medio
- 21h 9m
- PR unite (30g)
- 255
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un 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 rescript-lang/rescript
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
rescript-lang/rescript#9135 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
rescript-lang/rescript#9132 ·
I maintainer di solito rispondono entro 1 giorno
-
Printed type declarations drop @tag, @notUndefined and runtime annotationsForse già presa @cknitt l’ha presa oggi. Aperta
Difficoltà 4/5 1-2 giorni Idoneità per principianti 25/100
rescript-lang/rescript#9134 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
rescript-lang/rescript#9130 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 1-2 giorni Idoneità per principianti 55/100
rescript-lang/rescript#9122 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di rescript-lang/rescript
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 4 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
CanadaHonk/porffor#426 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
compiler-rt:asan llvm:transforms
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
llvm/llvm-project#236682 ·
I maintainer di solito rispondono entro 1 giorno