Skip to content

chore(nix): add flake#39

Merged
cmdoret merged 3 commits into
mainfrom
chore/nix-shell
Nov 3, 2025
Merged

chore(nix): add flake#39
cmdoret merged 3 commits into
mainfrom
chore/nix-shell

Conversation

@cmdoret
Copy link
Copy Markdown
Member

@cmdoret cmdoret commented Nov 3, 2025

No description provided.

@cmdoret cmdoret requested a review from supermaxiste November 3, 2025 10:52
@cmdoret cmdoret self-assigned this Nov 3, 2025
@supermaxiste
Copy link
Copy Markdown
Member

Some comments:

  1. If we add a Nix flake, I would add some explanation on the README how to use it. Can we temporarily add it to the make commands? Maybe like make nix-dev?

  2. The current flake is only Linux compatible. When running it on MacOS I get the message below. Let's write that Nix is only for Linux for now.

  3. CI: can we add an action that tries to build the Nix env when we change the flake?

error: flake 'git+file:///Users/stefan/Documents/kg-llm-interface?dir=tools/nix' does not provide attribute 'packages.aarch64-darwin.default' or 'defaultPackage.aarch64-darwin'

@cmdoret
Copy link
Copy Markdown
Member Author

cmdoret commented Nov 3, 2025

  1. Added explanation on Readme + recipe in justfile I figured I might do it in just directly, as we're migrating anyways
  2. Noted it's linux only for now (does it not work on aarch64 if you rm the lock file (tools/nix/flake.lock) and rebuild?
  3. added ci for building the devshell whenever a *.nix file is touched by a PR

@supermaxiste
Copy link
Copy Markdown
Member

I shall test the lock removal approach for another time. All looks good, excellent job 🚀

@cmdoret cmdoret merged commit 901b387 into main Nov 3, 2025
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