highlightjs/highlight.js

(C#) Support digit separators

Fermée

#4 258 ouverte le 9 mai 2025

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)JavaScript (3 497 forks)batch import
buggood first issuehelp welcomelanguage

Métriques du dépôt

Stars
 (22 960 étoiles)
Métriques de merge PR
 (Merge moyen 6m) (2 PRs mergées en 30 j)

Description

Describe the issue Digit separators are not supported: if you write 20_000_000 for example, 20 will be number-colored, while _000_000 will be treated as invalid.

Which language seems to have the issue? C#

Are you using highlight or highlightAuto? highlight

Sample Code to Reproduce

int i = 20_000_000;

Screenshot of broken syntax highlighting

Expected behavior the entirety of the number to be highlighted as a number, like on GitHub: Screenshot of correct syntax highlighting

Additional context None

Guide contributeur

Direction de recherche
Examinez le fichier de définition du langage C# (probablement src/languages/csharp.js). Le motif numérique actuel n'inclut pas les séparateurs de chiffres (tiret bas). Modifiez l'expression régulière pour autoriser les tirets bas entre les chiffres, par exemple /\d( ?\d)*/. Testez avec l'exemple de code '20 000 000'. Assurez vous que les autres formats numériques (hexadécimal, binaire, etc.) prennent également en charge les séparateurs, le cas échéant.
Stack technique
javascriptcsharp
Domaine
frontendtooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptRegular Expressions
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70