Skip to content

chore(nix): change rust toolchain to using fenix#225

Merged
mkqavi merged 1 commit into
mainfrom
push-topyzonzvmlz
May 19, 2026
Merged

chore(nix): change rust toolchain to using fenix#225
mkqavi merged 1 commit into
mainfrom
push-topyzonzvmlz

Conversation

@mkqavi
Copy link
Copy Markdown
Owner

@mkqavi mkqavi commented May 19, 2026

No description provided.

@mkqavi mkqavi added the dependencies Pull requests that update a dependency file label May 19, 2026
@mkqavi mkqavi self-assigned this May 19, 2026
@mkqavi mkqavi requested a review from Copilot May 19, 2026 12:41
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Switches the Nix dev shell's Rust toolchain from nixpkgs-provided components to the fenix flake, which provides more up-to-date and consistent Rust toolchains.

Changes:

  • Add fenix as a flake input (with nixpkgs follows) and wire it through to outputs.
  • Replace individual rustc/cargo/clippy/rustfmt/rust-analyzer packages with a single fenix.packages.${system}.stable.withComponents toolchain that also adds rust-src.
  • Update flake.lock to include the new fenix and rust-analyzer-src nodes.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
flake.nix Adds fenix input and builds the dev shell's Rust toolchain via fenix.packages.${system}.stable.withComponents.
flake.lock Locks the new fenix input and its rust-analyzer-src transitive input.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mkqavi mkqavi merged commit 442d5e5 into main May 19, 2026
7 checks passed
@mkqavi mkqavi deleted the push-topyzonzvmlz branch May 19, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants