Lint Rust demos with clippy

Aberta
#908 1 comentário 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
38/100
Tipo de issue
Funcionalidade
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Stack de tecnologia
javascript, rust
Domínio
cli, tooling

Direção de pesquisa

Comece em src/cli/main.js, por volta da linha 114, e rastreie como os demos são atualmente submetidos ao lint com servidores de linguagem. Identifique como os demos e seus arquivos Rust correspondentes são coletados e, em seguida, verifique se cada demo executa cargo clippy com o diretório de destino correto e limpa os arquivos copiados depois.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

enhancement rust

One thing that would be good to have is to check all Rust demos with clippy.
Clippy both ensures that they build correctly, but also acts as a linter.
Looking at the code it seems like we currently lint with language servers.
That is possible with rust-analyzer, but it's also wasteful compared to just running cargo clippy directly.

What I think should be done:

  • Loop through each demo
  • Copy all relevant Rust files into the demo
  • Run cargo clippy (make sure that target dir is set correctly)
  • Remove copied Rust files again
Linguagem predominante
JavaScript
Estrelas
846
Forks
91
Métricas de merge de PRs
Nenhum PR com merge em 30d

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de workbenchdev/Workbench

Todas as issues de workbenchdev/Workbench

Issues semelhantes

Mais issues de JavaScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.