microsoft / microsoft/TypeScript-TmLanguage
Somewhat misleading escape tokens in String.raw tagged templates
Nessuno ha ancora preso questa issue.
- Lingua principale
- TypeScript
- Stelle
- 471
- Fork
- 149
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
JavaScript and TypeScript Nightly version:
Code
// Please include a code snippet that demonstrates the issue
String.raw`--some-flag=C:\Some\Windows\Path` // It looks like the first letter of each dir is being escaped
String.raw`"This is\nJSON text\u2757"` // It looks like '\n' is being escaped
Example of how it looks in VSCode
Since String.raw is an EcmaScript standard, it's probably ideal that the grammar shows those string tokens without any escape tokens.
Important to note that backslashes shouldn't be ignored in the pattern, for example:
// ╭─ Template string opener
// ┴
String.raw`\`;
// ─┬─
// ╰─ Unclosed template string content
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
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 ispezionando le regole della grammatica TypeScript TextMate utilizzate per le template string e i tagged template, riproducendo gli esempi di String.raw in VS Code. Verifica i token di escape visualizzati rispetto al comportamento richiesto, incluso il caso di una template string non chiusa; il lavoro è completato quando il contenuto di String.raw non mostra più token di escape fuorvianti, mentre i backslash significativi continuano a influenzare il parsing.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- tooling
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100