Skip to content

Flake update and nixvim/hm nixpkgs pinning#125

Merged
UnstoppableMango merged 3 commits into
mainfrom
update
May 28, 2026
Merged

Flake update and nixvim/hm nixpkgs pinning#125
UnstoppableMango merged 3 commits into
mainfrom
update

Conversation

@UnstoppableMango

Copy link
Copy Markdown
Owner
  • Flake update
  • Crap. All crap

Copilot AI review requested due to automatic review settings May 28, 2026 16:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the flake lockfile (bumping direnv-instant, home-manager, nixd, nix-direnv, nix-vscode-extensions, nixpkgs, and nixvim) and changes the home-manager and nixvim flake inputs so they no longer follow the root nixpkgs, letting each provide its own. The PR also accidentally adds two extensionless scratch files (3 and 4) at the repository root.

Changes:

  • Stop pinning home-manager and nixvim to the root nixpkgs in flake.nix, adding comments explaining the rationale.
  • Refresh flake.lock to match, introducing a separate nixpkgs_3 node for nixvim and reusing the freed nixpkgs slot for home-manager's own pin.
  • Add two extensionless files (3, 4) at the repo root containing alternative flake variants — these appear to be unintended.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
flake.nix Removes inputs.nixpkgs.follows = "nixpkgs" for home-manager and nixvim, replacing with comments.
flake.lock Lock updates for many inputs plus restructuring of nixpkgs nodes to reflect un-pinned hm/nixvim.
3 Appears to be an accidentally committed scratch copy of an alternative flake.nix; should be removed.
4 Identical scratch copy of 3; should be removed.

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

Comment thread 3 Outdated
Comment thread 4 Outdated
@UnstoppableMango UnstoppableMango merged commit daafc1b into main May 28, 2026
2 checks passed
@UnstoppableMango UnstoppableMango deleted the update branch May 28, 2026 17:44
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.

2 participants