gleam-lang/gleam

LSP: Show an error message on an invalid named module

Open

#3.972 aberto em 9 de dez. de 2024

Ver no GitHub
 (5 comments) (0 reactions) (0 assignees)Rust (960 forks)batch import
good first issuehelp wanted

Métricas do repositório

Stars
 (21.417 stars)
Métricas de merge de PR
 (Mesclagem média 10d 19h) (69 fundiu PRs em 30d)

Description

Hi!

Currently, when the LSP opens a Gleam file whose name is incorrect (something like my-module.gleam for instance), nothing happens. It could be nice to let the LSP indicates it as a bug (because gleam build will fail) with a message like "my-module is not a valid name for a Gleam module. You should switch to my_module instead."

Guia do colaborador

Direção de pesquisa
A issue sugere adicionar um diagnóstico para nomes de módulo inválidos no Gleam LSP. Investigue a base de código da crate LSP, particularmente o arquivo onde os diagnósticos são emitidos (por exemplo, a crate `lsp` ou `ls`). Verifique como os diagnósticos existentes são implementados, como para erros de sintaxe. O objetivo é adicionar uma verificação para a validade do nome do módulo e emitir um erro com uma sugestão como 'my module' em vez de 'my module'. Revise os testes e a documentação existentes para adicionar novos tipos de diagnóstico.
Pilha de tecnologia
rust
Domain
developer experience
Tipo Issue
Funcionalidade
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.
Meio dia
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
Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70