highlightjs/highlight.js

(Crystal) formats integer divison incorrectly

Open

#3.730 geöffnet am 3. März 2023

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (3.497 Forks)batch import
buggood first issuehelp welcomelanguage

Repository-Metriken

Stars
 (22.960 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Describe the issue Using // in crystal will make the highlighter mark the rest of the file. In crystal is // used for integer division. Code:

def something()
    something // 4
end

image

Which language seems to have the issue? Crystal

Are you using highlight or highlightAuto? highlight

Sample Code to Reproduce

def something()
    something // 4
end

image

Expected behavior The expected behavior is to not mark the rest of the code with the color green.

Additional context Nope

Contributor Guide

Research-Richtung
Untersuchen Sie die Crystal Sprachdefinitionsdatei in highlight.js (wahrscheinlich src/languages/crystal.js). Das Problem ist, dass der Divisionsoperator // als Kommentaranfang erkannt wird. Finden Sie den für die Kommentarerfassung verantwortlichen regulären Ausdruck und passen Sie ihn an, um korrekt zwischen Ganzzahldivision und Kommentaren zu unterscheiden. Erwägen Sie, ein Muster für den Divisionsoperator vor dem Kommentarmuster hinzuzufügen.
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
JavaScriptregex
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70