beautifier/js-beautify

Beautification problem with JS Single Line Comment <!--

Offen

#1.835 geöffnet am 31.08.2020

Auf GitHub ansehenContributor Guide
 (10 Kommentare) (0 Reaktionen) (1 zugewiesene Person)JavaScript (1.460 Forks)batch import
good first issuelanguage: javascripttype: bug

Repository-Metriken

Stars
 (8.342 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

In Javascript <!-- is treated as a single-line comment just like //.

js-beautify works as expected when the line starts with <!--:

Input: <!--alert(1)commented

Output: <!--alert(1)commented

Otherwise it will break the syntax:

Input: alert(1)<!--commented

Output: alert(1) < !--commented

Contributor Guide

Research-Richtung
Untersuche den JavaScript Tokenizer in js beautify, um zu finden, wo `<! ` behandelt wird. Suche nach Mustern, die fälschlicherweise Leerzeichen nach `alert(1)` hinzufügen, wenn `<! ` folgt. Korrigiere die Tokenisierung oder Formatierungslogik, um `<! ` als einzeiligen Kommentar zu erhalten.
Tech Stack
javascript
Domain
tooling
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75