Skip to content

Pin nixpkgs by committing root flake.lock - #2

Merged
dasmatus merged 2 commits into
mainfrom
copilot/fix-review-comment-4052712068
Sep 19, 2026
Merged

dasmatus merged 2 commits into
mainfrom
copilot/fix-review-comment-4052712068

Conversation

Copilot AI commented Sep 19, 2026 •

Copy link
Copy Markdown

What Changed

The review comment flagged that flake.nix referenced nixpkgs-unstable without a committed root lockfile, making first-time nix develop non-reproducible. This update addresses only that comment by adding the repository-root flake.lock.

  • Lockfile added
    • Added /flake.lock at repo root to pin the nixpkgs input used by /flake.nix.
  • Scope kept surgical
    • No edits to flake.nix, build logic, or other repo files.
{
  "nodes": {
    "nixpkgs": {
      "locked": {
        "rev": "044bfe75bfe4c7bbe043dc17b5e42ea823b84a09"
      }
    }
  }
}

Why

Without a committed root flake.lock, each environment resolves whatever nixpkgs-unstable points to at that moment, producing drift and dirty working trees. Committing the lockfile makes dev-shell resolution deterministic for contributors and CI.

UI Changes

No UI changes.

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes
  • I included a video for animation/interaction changes

Co-authored-by: dichhead <330471626+dichhead@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix code for review comment 4052712068 Pin nixpkgs by committing root flake.lock Sep 19, 2026
Copilot AI requested a review from dasmatus September 19, 2026 09:09

@dasmatus dasmatus left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Flake.lock added

@dasmatus
dasmatus marked this pull request as ready for review September 19, 2026 09:10
@dasmatus
dasmatus merged commit 9913a7f into main Sep 19, 2026
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