-
-
Notifications
You must be signed in to change notification settings - Fork 508
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
Add nix files and install instructions #477
Conversation
Ignorant question: I know very little about Nix. What's special about this compared to nixpkgs? |
The primary benefit of this is that on nixos, I can run |
Hey! Thanks for the work here 🚀 Totally happy to get this merged if you could fix up the whitespace changes? The diff is a bit noisy |
I'd be happy to clean this up if you wanted merge this, I also use Nix and occasionally contribute to nixpkgs. I will say though, that any user of NixOS (which I am) can always install the necessary build dependencies with As an example of what it can be used for, Helix uses it quite extensively to setup identical dev environments with a number of extra tools. The benefits to adding it to the |
yes please @patricksjackson, that would be fantastic! |
913e9ed
to
d2ac2df
Compare
I've cleaned the diffspam in the readme 🙂 |
Those clippy warnings are not me - they happen on current main as well |
Thank you! Yeah don't worry they're just because there's a new Rust out |
@Sciencentistguy I'm seeing this error on my Macbook. Should it have worked?
|
Considering flakes are technically experimental and will be for the foreseeable future, it's probably a good idea to add a backwards compatible In addition, adding a note somewhere to the docs about how to enter the development shell would help let people know that it exists. |
@charego can you try replacing I don't have access to a Mac to confirm, but that should work. |
Sorry I keep responding in separate posts, but I'd also suggest adding a Github Action that runs Adding caching with Cachix would be nice but probably not necessary yet. |
flake-compat is obsolete since nix added (builtins.getFlake (toString ./.))
.devShells
.${builtins.currentSystem}
.default |
d2ac2df
to
cebe488
Compare
Oh interesting, I didn't know about But it's only available if flakes are enabled, so it doesn't work for writing a backwards compatible |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tbh I'm happy to get this in as-is, even if there's a little more we could do. Thanks for the work @Sciencentistguy <3
Add a flake.nix to facilitate easier development on NixOS, and to allow atuin to be installed using
nix profile
. Also add a nix section to the install instructions in the readme