-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Remove nix cruft #475
Remove nix cruft #475
Conversation
Nice! I don't think I understand what a Some questions:
|
yep, exactly
sorry, old habits die hard
yea,
|
Whoops I'm silly, I was confused and thought this issue was in github.com/casey/just 😅 |
Sweet, looks good. Is this ready to merge? |
yep! thanks for the feedback. |
Awesome, this is great! I really like how simple it makes getting started. |
So, I had added a flake on the fly for development and building using Nix, and haven't kept up with the project as I should have, and it doesn't use an explicit Rust version, nor does it build correctly. After playing with this, a few reasons to change the current setup:
shell.nix
is sufficient because it addresses all Nix users irrespective of version/setup: justnix-shell
in the project directory and you're set up for development, which is the whole point.Happy to nuke the Nix stuff if you think it's annoying, but the only maintenance here will be bumping rust versions as needed.