Nix support - #41
Draft
AamodJ wants to merge 9 commits into
Draft
Conversation
AamodJ
marked this pull request as draft
July 31, 2026 21:18
BarbUk
reviewed
Aug 2, 2026
Comment on lines
+5
to
+14
| mkShell { | ||
| packages = with nixpkgs; [ | ||
| shellcheck | ||
| uv | ||
| python3 | ||
| rustup | ||
| zizmor | ||
| rumdl | ||
| typos | ||
| prek |
Owner
There was a problem hiding this comment.
Those are all dev deps, used only with pre-commit or prek to contribute to the project
Owner
|
Thanks @AamodJ !
As I don't use nix, I'm open to changes that can help with integrating snippy with nix. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hello!
This PR is to add nix support to this project with a simple flake providing some home manager module options. I suggest keeping a separate branch to work on nix support. And I would like to maintain this branch for the project.
By default, setting
programs.snippy.enable = true;will install both wayland and x11 dependencies (since we don't know the user's environment), and we provide options to enable wayland or x11 exclusively (in effect this will not install dependencies for the other).Next steps:
flake.lockto point it to the latest commitnixfmtin CI pipeline and pre-commit hooks to ensure code consistencyLICENSE.md. Currently only the script is installed so the license is not distributed. I will add this in a future commitSuggestions are welcome.