Skip to content

Install a couple of LLM tools#98

Merged
stehessel merged 2 commits intomasterfrom
llms
Apr 9, 2026
Merged

Install a couple of LLM tools#98
stehessel merged 2 commits intomasterfrom
llms

Conversation

@porridge
Copy link
Copy Markdown
Collaborator

@porridge porridge commented Apr 8, 2026

No description provided.

@porridge porridge requested a review from stehessel April 8, 2026 10:32
@porridge
Copy link
Copy Markdown
Collaborator Author

porridge commented Apr 8, 2026

This "Note" looks scary, is this compatible with how we're doing things? 🤔

Copy link
Copy Markdown
Collaborator

@stehessel stehessel left a comment

Choose a reason for hiding this comment

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

This "Note" looks scary, is this compatible with how we're doing things? 🤔

It should be compatible. They are essentially warning of using an optimization where the base tree is pinned to something other than they used to build the packages. Without follows (the optimization) the base tree is duplicated to match the exact versions. However, in practice it's probably best to rely on the binary cache anyways.

Comment thread flake.nix Outdated
@porridge porridge requested a review from stehessel April 9, 2026 06:28
@stehessel stehessel merged commit 685c280 into master Apr 9, 2026
3 checks passed
@stehessel stehessel deleted the llms branch April 9, 2026 07:39
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