Skip to content

chore(rust): eliminar los avisos de compilación por plataforma (#246) - #247

Merged
egdev6 merged 2 commits into
mainfrom
chore/246-windows-warnings
Nov 6, 2026
Merged

egdev6 merged 2 commits into
mainfrom
chore/246-windows-warnings

Conversation

@egdev6

@egdev6 egdev6 commented Nov 6, 2026

Copy link
Copy Markdown
Owner

Issue relacionada

Closes #246

Alcance

Acota con #[cfg(...)] los símbolos que solo se usan en algunas plataformas, para que cargo compile sin avisos en el crate quorb. Sin #[allow(...)] generales y sin cambio de comportamiento.

  • providers/claude.rs, providers/gemini.rs: se elimina use std::path::Path (sin uso en ninguna plataforma). KeychainOutcome::Session/TimedOut y sus ramas del match pasan a cfg(target_os = "macos"), el único lugar donde se construyen.
  • detail_placement.rs: rooted_origin, to_physical y SETTLE se importan solo en Linux, donde los usa UiSurface.
  • cli_process.rs: let mut child → let child (run_pipelined recibe el hijo por valor).
  • login.rs: path_with solo en Windows; path_with_current en Windows y tests; split_login_command en Linux y tests.
  • wm_compat.rs: with_display_backend en Linux y tests.
  • wsl/process.rs: SCRIPT_READ_PID solo en tests (producción usa SCRIPT_READ_IDENTITY).
  • uninstall.rs: el helper de borrado de datos de NSIS (Plan, purge, is_link, owned_target, variantes de Failure e imports) en Windows y tests. Estos 6 avisos aparecían en Linux.

Validación

  • Linux: cargo check --all-targets --locked sin avisos (antes: 13 en la lib y 5 en tests).
  • cargo test --locked: 509 passed, 5 ignored, 0 failed.
  • cargo fmt --check e node scripts/i18n-inventory.mjs --check: OK.
  • Windows y macOS: no se pueden compilar en local; los verifico en los logs del CI de esta PR.

@egdev6
egdev6 merged commit f08203b into main Nov 6, 2026
5 checks passed
@egdev6
egdev6 deleted the chore/246-windows-warnings branch November 6, 2026 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore(rust): eliminar los avisos de compilación en Windows

1 participant