highlightjs/highlight.js

(Crystal) formats integer divison incorrectly

Open

#3.730 aberto em 3 de mar. de 2023

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)JavaScript (3.497 forks)batch import
buggood first issuehelp welcomelanguage

Métricas do repositório

Stars
 (22.960 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Examine o arquivo de definição da linguagem Crystal no highlight.js (provavelmente src/languages/crystal.js). O problema é que o operador de divisão // está sendo correspondido como o início de um comentário. Encontre a expressão regular responsável pela detecção de comentários e ajuste a para diferenciar corretamente entre divisão inteira e comentários. Considere adicionar um padrão para o operador de divisão antes do padrão de comentário.
Pilha de tecnologia
javascript
Domain
tooling
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptregex
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70