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

Use nix-env in ci #81

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Use nix-env in ci #81

wants to merge 5 commits into from

Conversation

mmlb
Copy link
Member

@mmlb mmlb commented Mar 21, 2022

Somehow I didn't think to use nix-env in CI until reading a snippet of someone doing so in a hackernew or mabye lobste.rs comment. This makes it so much nicer to use imo. I also got rid of the nix-shell in the formatters.sh shebang to make it more useful for users, no need to tie it to nix-shell and make it hard/unusable to people when CI will catch any diffs due to different tool versions.

This is easier to use instead of having to go through nix-shell for each
command. This also gets rid of the nix-shell in the shebang for formatters.sh
so that we don't require nix-shell for its use. Different versions of tools
will be caught by CI anyway so its fine to be lenient for users.
@mmlb mmlb force-pushed the use-nix-env-in-ci branch 3 times, most recently from b0c40e2 to b45a632 Compare March 21, 2022 20:03
@mmlb mmlb marked this pull request as draft March 21, 2022 20:37
@mmlb mmlb force-pushed the use-nix-env-in-ci branch 6 times, most recently from 23864e6 to 078a3d1 Compare March 22, 2022 18:57
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