microsoft / microsoft/TypeScript-TmLanguage

HTML comments are not considered comments

Aperta
#781 2 commenti 2 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Suggestion
Lingua principale
TypeScript
Stelle
471
Fork
149
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

In JavaScript, HTML comments are a valid replacement for single line comments.
They are considered comments in Node.js, inline script tags, external js file, etc.

<!-- No comment.
// Comment.

Even GitHub doesn't seem to understand this syntax 🤔

Your grammar does not seem to support them, and mistakes them as a syntax error.

Link to the relevant part of the spec

In the spec this syntax appears under additional syntax for web browsers, yet you do support the other additional syntax in that section, such as legacy octal numeric literals and octal string escapes, but not HTML comments.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

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.

Direzione di ricerca

Inizia individuando le regole della grammatica TextMate di TypeScript per i commenti JavaScript e confrontale con la sezione di ECMAScript sui commenti simili all’HTML collegata nell’issue. Verifica la grammatica con l’esempio di commento HTML mostrato e conferma che non venga più segnalato come errore di sintassi.

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

Valutazione

Stack tecnologico
javascript, 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
38/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.