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 #295

Merged
merged 2 commits into from
Oct 16, 2023
Merged

Add flake.nix #295

merged 2 commits into from
Oct 16, 2023

Conversation

bouk
Copy link
Contributor

@bouk bouk commented Sep 15, 2023

This adds a flake.nix with an overlay and a package. The nixpkgs is pinned to the one used by the rest of the repository, so its nixpkgs version is not managed separately.

After this PR crate2nix can be run by calling nix run github:nix-community/crate2nix. Crate2nix can also be used via the overlay by using crate2nix.overlays.default

@kolloch
Copy link
Collaborator

kolloch commented Sep 15, 2023

Hi @bouk, thank you!

could you please reformat it with https://github.com/nix-community/crate2nix/blob/master/nixpkgs-fmt.sh?

while it is a quite minimal wrapper around existing config, it would be nice if this was checked in CI so that we don’t break it accidentally.

@tilpner
Copy link
Contributor

tilpner commented Sep 20, 2023

It would be nice if this could export tools.nix in some way. Not sure where it fits in exactly, maybe that can wait until after flake-schemas is a thing, and you can make up attribute semantics.

This adds a flake.nix with an overlay and a package. The nixpkgs is pinned to the one used by the rest of the repository, so its nixpkgs version is not managed separately.
Also add formatter to flake so you can do `nix fmt`
@kolloch
Copy link
Collaborator

kolloch commented Oct 16, 2023

Thanks!

@kolloch kolloch merged commit eed2f45 into nix-community:master Oct 16, 2023
1 of 2 checks passed
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.

3 participants