-
Notifications
You must be signed in to change notification settings - Fork 44
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
dev: add nix flake environment #5297
Conversation
bd4d85c
to
ed1c091
Compare
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.
I'm not a Nix expert but it seems to work fine 👌
I just spotted the loss of autocompletion for the cargo
commands.
ed1c091
to
c7cb556
Compare
For the autocompletion on cargo commands, I'm searching but I'm not quite sure why it's doing this. I've taken into account your suggesstions, just changed the nodePackages with a version that ensure the same nodejs version is used accross all "globally" installed packages. |
c7cb556
to
2f96ac0
Compare
- Implement flakes with development dependencies - Add a `shell.nix` that leverages edolstra/flake-compat
2f96ac0
to
9aebedc
Compare
The only changes, since the validation of @multun, is the addition of a shell.nix that leverages https://github.com/edolstra/flake-compat (edolstra is the creator of nix). It simply is a retrocompatibility layer. |
I've added a flake-ified version of https://github.com/flomonster/osrd-nix-config.
You can fire up a dev shell by using
nix develop
. (You might need to enable experimental flags on nix to use flakes).This is still a work in progress, here are some elements that should be discussed:
nix/
folder.scripts/
folder as commands that can be directly called with their names (./script/generate-infra.sh
will begenerate-infra
directly in the dropped shell).