microsoft / microsoft/TypeScript
Double slash comment `//` will be wrongly removed in the trailing position of the object attribute
Aperta
Nessuno ha ancora preso questa issue.
Domain: Comment Emit
Help Wanted
Possible Improvement
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.4k
- Merge medio
- 1g 19h
- PR unite (30g)
- 117
Descrizione
🔎 Search Terms
- comment removal
- removeComments
🕗 Version & Regression Information
- This is the behavior in every version I tried.
⏯ Playground Link
💻 Code
export const foo = {
foo: 'foo', // this comment will be removed, however this should be kept
bar: 'bar' // this comment will be kept, works well
}
🙁 Actual behavior
export const foo = {
foo: 'foo',
bar: 'bar' // this comment will be kept, works well
}
🙂 Expected behavior
export const foo = {
foo: 'foo', // this comment will be removed, however this should be kept
bar: 'bar' // this comment will be kept, works well
}
Additional information about the issue
No response
Guida per i contributori
Apri 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.
Direzione di ricerca
Inizia tracciando il comportamento di removeComments indicato nell’issue e riproduci il caso nel TypeScript Playground collegato. Confronta la gestione dei commenti per l’attributo finale dell’oggetto con e senza una virgola successiva. Il lavoro è completato quando il commento foo mostrato viene preservato mentre il commento bar esistente rimane invariato, con un test di regressione che copra l’esempio.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- compilers
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 48/100