Skip to content

nix: bump flake.lock, fix nitpick in drv, add .envrc#88

Open
es-sai-fi wants to merge 1 commit intoJayanAXHF:mainfrom
es-sai-fi:main
Open

nix: bump flake.lock, fix nitpick in drv, add .envrc#88
es-sai-fi wants to merge 1 commit intoJayanAXHF:mainfrom
es-sai-fi:main

Conversation

@es-sai-fi
Copy link

No description provided.

@es-sai-fi es-sai-fi marked this pull request as ready for review March 12, 2026 20:07
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 12, 2026

Greptile Summary

This PR makes three small, related improvements to the Nix packaging setup: it bumps the pinned nixpkgs revision to a more recent nixos-unstable commit, adds a standard direnv .envrc to auto-activate the flake dev-shell when entering the directory, and cleans up a redundant with lib; scope in package.nix.

  • flake.lock: Routine nixpkgs bump (2fc653962dc67aa); narHash and lastModified are updated consistently.
  • .envrc: Adds standard watch_file directives for flake.lock and flake.nix so direnv re-evaluates the shell when either changes, followed by use flake to activate the dev environment. This is the idiomatic pattern for Nix flake projects.
  • package.nix: Removes the outer with lib; scope from the meta block. The only attribute that needed it was license, which already has its own with lib.licenses; scope — so the removal is correct and improves clarity by reducing implicit namespace pollution.

Important Files Changed

Filename Overview
.envrc New direnv config that watches flake.lock and flake.nix and activates the Nix flake environment; standard and correct pattern for Nix flake projects.
flake.lock Routine bump of the pinned nixpkgs revision (nixos-unstable) to a newer commit; narHash and lastModified updated consistently.
package.nix Removes unnecessary top-level with lib; scope from the meta block; lib.licenses is still correctly referenced explicitly in the license line, so the change is safe.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Developer enters project directory] --> B{direnv installed?}
    B -- Yes --> C[.envrc is loaded automatically]
    B -- No --> G[Manual: nix develop]
    C --> D[watch_file flake.lock\nwatch_file flake.nix]
    D --> E[use flake]
    E --> F[Nix evaluates flake.nix\ndevShells.default via shell.nix]
    F --> H[Dev environment activated\ntools & dependencies available]
    G --> H
    D --> I{flake.lock or flake.nix changed?}
    I -- Yes --> C
Loading

Last reviewed commit: d42fa2d

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