No local toolchain pin: dev-box stable (1.99.0) already disagrees with CI's 1.99.1 — clippy fires chunks_exact_to_as_chunks on one and not the other

Abierto Apto para principiantes
  • ¿Algo no está bien?
#146 2 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
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Nueva funcionalidad
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
rust
Área
build-system, tooling

Línea de trabajo

Lee .github/workflows/ci.yml y la advertencia de cobertura relacionada con la toolchain de xtask coverage. Añade el pin propuesto en rust-toolchain.toml para que coincida con CI 1.99.1. Se considerará terminado cuando cargo run -p xtask -- ci use localmente la misma toolchain que CI.

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

Descripción

ci

Evidence (2026-11-13). CI pins toolchain: 1.99.1 (.github/workflows/ci.yml, the #135 fix), but the repo carries no rust-toolchain.toml, so a dev box runs whatever its rustup default is. On this box the default stable is 1.99.0, and the two toolchains already disagree:

  • cargo run -p xtask -- ci on 1.99.0 fails clippy on crates/engine/src/catalog/mask.rs:311 and :1292 (clippy::chunks_exact_to_as_chunks, -D warnings), while the same code passes CI's clippy 1.99.1 — the lint fires on .0 and not on .1.
  • xtask coverage already warns about the mismatch: "measured with rustc 1.99.0, not the rustc 1.99.1 the floors were measured with" (and the floors are toolchain-bound: 1.99.0 → 81.9/79.0 vs 1.99.1 → 81.9/78.9).

So a contributor can have a locally-red gate that CI ignores, or a locally-green gate CI rejects, purely from the default toolchain. Proposal: add rust-toolchain.toml pinned to 1.99.1 (matching #135's CI pin and the coverage floors' toolchain), so cargo run -p xtask -- ci means the same thing everywhere; bump the pin and CI's toolchain: in the same change whenever the floors are re-measured.

Lenguaje dominante
Rust
Estrellas
11
Forks
1
Merge medio
13 h 16 min
PR fusionados (30 d)
38

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 aliasfoxkde/FireCraft

Todos los issues de aliasfoxkde/FireCraft

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.