Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

ci: lint-enforce expect-with-reason

Abierto
  • ¿Algo no está bien?
#2,111 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
48/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
rust
Área
ci-cd, testing

Línea de trabajo

Start by reading AGENTS.md and inspect Rust lint attributes across the workspace, especially the named crates and binary main.rs files. The issue provides rg commands to reproduce the counts; review each suppression and add a reason or use expect where it is valid across feature configurations. Done when both workspace Clippy lints are enabled and CI passes with all suppressions addressed.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

AGENTS.md asks for #[expect(clippy::<lint>, reason = "...")] when silencing a lint; neither half is checked.
Nothing stops an #[allow], and nothing stops an #[expect] without a reason.

IronRDP and IronVNC set clippy::allow_attributes = "warn" in [workspace.lints.clippy]; this workspace does not.
Enabling both lints makes each half fail CI under -D warnings:

[workspace.lints.clippy]
allow_attributes = "warn"
allow_attributes_without_reason = "warn"

Existing sites need a sweep first.
Counts at c6e0595e over --type rust:

Pattern Sites Files
#[allow(...)] (outer) 104 44
#![allow(...)] (inner, crate-level) 100 74
#[expect(...)] without reason 25 18
#![expect(...)] without reason 0 0

Of the 204 allow sites, 34 already carry a reason; the rest need one or a switch to expect.
Outer #[allow] sites by area: devolutions-gateway 50, crates/devolutions-pedm-shared 19, devolutions-agent 11, devolutions-session 11, the remainder spread across seven crates and tools.
#[expect] sites without reason by area: crates/win-api-wrappers 11, crates/devolutions-pedm 7, devolutions-gateway 4, crates/pedm-simulator 2, jetsocat 1.

Most inner #![allow] sites are unused_crate_dependencies and clippy::unwrap_used at the top of devolutions-gateway/tests/*.rs and print_stdout/print_stderr in binary main.rs files.
Crate-level suppressions that are only sometimes fulfilled, such as unused_crate_dependencies under feature combinations, may need to stay #![allow] with a reason rather than become #![expect], which would trigger unfulfilled_lint_expectations in the other configuration.

Reproduce the counts with rg -c -U '#!?\[allow\([^\]]*\)\]' --type rust and rg -c -U --pcre2 '#\[expect\((?:(?!reason)[^\]])*\)\]' --type rust.

[!NOTE]
Human-tuned, LLM-assisted content.

Lenguaje dominante
Rust
Estrellas
164
Forks
30
Merge medio
1 d 13 h
PR fusionados (30 d)
84

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de Devolutions/devolutions-gateway

Todos los issues de Devolutions/devolutions-gateway

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.