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.

Sign up for GitHub

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

Jump to bottom

Implement Nix flakes ❄️ #40

Merged
merged 5 commits into from
Jul 21, 2022
Merged

Implement Nix flakes ❄️ #40

merged 5 commits into from
Jul 21, 2022

Conversation

sioodmy
Copy link
Contributor

@sioodmy sioodmy commented Jul 19, 2022

This pull request implements Nix flake, which makes installation, development and distribution easier.

for user:

  • nix run github:siduck/st - Compiles and runs st from the latest commit, very useful when someone wants to try it out. The user doesn't have to worry about installing dependencies or compiling.
  • Users can easily install this fork through the flake

for contributors:

  • nix develop - puts you in a shell with all the necessary dependencies
  • nix build - compiles without the need of installing dependencies
  • nix run - compiles and runs without the need of installing dependencies
  • Zero maintenance - you don't have to touch anything when a new version comes out (unless there are new dependencies) and users will still receive updates

@siduck
Copy link
Owner

siduck commented Jul 20, 2022

thx a lot for the pr! @sioodmy but i dont use nixos so cant test this, @javacafe01 does this work?

javacafe01
javacafe01 suggested changes Jul 21, 2022
View reviewed changes
flake.nix Show resolved Hide resolved
@sioodmy
Copy link
Contributor Author

sioodmy commented Jul 21, 2022

is it working now?

javacafe01
javacafe01 approved these changes Jul 21, 2022
View reviewed changes
Copy link

@javacafe01 javacafe01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works great

@javacafe01
Copy link

@siduck Good to merge

@siduck siduck merged commit 71f9529 into siduck:main Jul 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@siduck siduck siduck approved these changes

@javacafe01 javacafe01 javacafe01 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sioodmy @siduck @javacafe01