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

Introduce nix tooling for reproducable dev envirionments #36

Merged
merged 2 commits into from
Aug 4, 2024

Conversation

Sntx626
Copy link

@Sntx626 Sntx626 commented Aug 2, 2024

Added a nix-shell to the project.
This allows developers to ephemrally install pinned packages to easily get a working dev env.

  • feat: added nix tooling

This only has an effect if developers use nix, it changes nothing about the project for anyone else.

@dezoito dezoito merged commit a903cde into dezoito:main Aug 4, 2024
1 check passed
@dezoito
Copy link
Owner

dezoito commented Aug 4, 2024

Thank you for your interest in the project.

The PR is merged, but I'll ping you to update that file in case new dependencies are added, ok? ;)

If you don't mind me asking, why did you include gnome.libsoup? Is it something you commonly use in developing Rust apps?

@Sntx626
Copy link
Author

Sntx626 commented Aug 4, 2024

Great!

I'll ping you to update that file in case new dependencies are added, ok? ;)

No problem at all! Feel free to ping me if something about the nix code needs to be changed or dependencies added/removed.

If you don't mind me asking, why did you include gnome.libsoup? Is it something you commonly use in developing Rust apps?

I figured out the required depencies through trial and error.
Since you asked, I looked into it a bit more and figured out that libsoup is a depency of the project (c.f. cargo tree | grep soup). However it is also added by the webkitgtk dependency, so we can forego adding gnome.libsoup specifically.

I also noticed that I forgot to add yarn as a depency. It seems that I still was in another devshell providing yarn when creating this flake...

I'll add a follow-up PR removing gnome.libsoup and adding yarn.

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