rust-lang/rustfmt

Option to ignore modules that cannot be resolved

Open

#5.609 aberto em 18 de nov. de 2022

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)Rust (760 forks)batch import
C-feature-requestP-lowgood first issue

Métricas do repositório

Stars
 (4.893 stars)
Métricas de merge de PR
 (Mesclagem média 10d 21h) (8 fundiu PRs em 30d)

Description

Hi

I am working on async-stripe which generates rust files and run into frequent errors in my automatic CI pipeline when they (the upstream openapi) remove parts of the api surface, as those modules are still referenced in the manual written mod.rs files. The automatic codegen runs, rustfmt is applied, and it fails due to missing files.

Now, I could (and will in the mean time) just set the rustfmt step as optional and apply it manually as these cases need to be manually intercepted, but I would love it if that, rather than eagerly fail, there was an opt-in option to simply prune the modules that now no-longer exist such that it can recover and format the ones that do.

Formatting is then also checked in the subsequent PR that is generated, but it would be nice to have the PR be created with formatting applied at a best-effort level.

See https://github.com/arlyon/async-stripe/actions/runs/3493571680/jobs/5848542195

Cheers,

Alex

Guia do colaborador

Direção de pesquisa
Investigue o código fonte do rustfmt, especificamente a lógica de resolução de módulos em src/mod.rs ou arquivos similares. Veja como ele atualmente lida com arquivos de módulo ausentes e onde os erros são gerados. Considere adicionar uma nova opção de CLI, por exemplo, ' ignore unresolved modules', que permita que a formatação prossiga mesmo quando os módulos não puderem ser resolvidos. A implementação deve capturar o erro e pular a formatação para o módulo ausente. Consulte as opções de CLI existentes (por exemplo, edition) para o padrão. Além disso, certifique se de que qualquer PR inclua testes para esta nova opção.
Pilha de tecnologia
rust
Domain
tooling
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.
3
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
RustGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65