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

Add flake.nix #33

Closed
wants to merge 1 commit into from
Closed

Add flake.nix #33

wants to merge 1 commit into from

Conversation

expede
Copy link
Contributor

@expede expede commented Apr 21, 2024

This PR is extremely optional, but I figured I'd contribute this while I had the hood up for #32

In short: I added a flake.nix for any other Nix users show up. This doesn't impact non-Nix users in any way, but gives anyone with Nix installed an instant, correctly configured dev environment.

@vmx
Copy link
Member

vmx commented Apr 22, 2024

There used to be a flake.nix file in other IPLD/Multiformat repos I maintain. My problem is that I know almost nothing about Nix and to me it seems like the lock file would also needed to be updated from time to time.

I can neither really review the change, not maintain it. Hence I would prefer if we wouldn't need to check Nix specific things into this repo. If there's any other way to make the life of Nix users easier, I'm happy to merge any PRs in regards to that.

@expede
Copy link
Contributor Author

expede commented Apr 22, 2024

@vmx Okay no prob 👍

If there's any other way to make the life of Nix users easier, I'm happy to merge any PRs in regards to that.

It's pretty much just the flake.nix. Being open source, Nix people would need to contribute updates to this repo to help each other out (like this PR), but it sounds like that's not how you want to run this repo which is totally fine!

@expede expede closed this Apr 22, 2024
@expede expede deleted the nix-flake branch April 22, 2024 18:13
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