Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: adds flake.nix with home manager support #9

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

higherorderfunctor
Copy link
Contributor

@higherorderfunctor higherorderfunctor commented Jun 30, 2024

Removes the original example Nix configuration I had contributed and adds a flake.nix to the repo with more thorough installation steps. A flake.nix in the repository allows Nix users to install directly from GitHub.

This is in response to the new comments on #2 (comment).

The following files are added:

  • flake.nix - Entry point that the Nix package manager looks for when installing from GitHub.
  • flake.lock - Nix is declarative and enforces pinned dependencies. This shouldn't need to be updated very often if at all. The only dependency is on nixpkgs and the instructions have the user override it with whatever version they use.
  • nix/default.nix - Creates the Nix plugin package.
  • nix/generate-config.nix - Helper to convert YAML configurations into a Nix source file.
  • nix/home-manager.nix - Nix is typically focused on system configurations whereas home manager focuses on per-user configurations. This adds options the user can use to install and configure the plugin with Nix expressions. It handles converting Nix expressions back into a YAML configuration and running build.py to produce the final tmux configuration.

I updated README.md by removing the old example and put in more thorough examples using the flake.nix. I also corrected a few linting errors for indented code blocks under a list and HTML elements.

I'm also happy to be a CODEOWNER for these files to keep them maintained if wished. If you want me to add, just let me know!

@higherorderfunctor
Copy link
Contributor Author

Couple updates:

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