Skip to content

Stricter lints, doc and dependency checks - #13

Merged
thomasjm merged 6 commits into
mainfrom
stricter-lints
Nov 15, 2026
Merged

thomasjm merged 6 commits into
mainfrom
stricter-lints

Conversation

@thomasjm

Copy link
Copy Markdown
Contributor

Adds more checks to nix flake check and fixes what they found:

  • Docs build with warnings as errors (fixed 16 broken links).
  • rust_2018_idioms, unused_qualifications and unsafe_op_in_unsafe_fn.
  • Clippy's cast lints: casts that could truncate or change sign now use checked conversions, or say why they're fine.
  • Every unsafe block needs a // SAFETY: comment. Two were removed as unnecessary, and FdTransport::from_raw_fd now owns its fd from the start (it used to leak it on errors).
  • cargo-deny (licenses, banned crates, sources; deny.toml) and cargo-audit with a pinned advisory-db flake input. Updated tokio and rustix past unsound-code advisories; the fuser one doesn't apply since we build it without libfuse.

@thomasjm
thomasjm merged commit d943cc4 into main Nov 15, 2026
2 checks passed
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.

1 participant