-
Notifications
You must be signed in to change notification settings - Fork 7
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
docs: add how to use via nixpkgs #387
Conversation
Thanks for your contribution, @kachick, and for adding the nix package definition! I like the idea of mentioning it in I'll review the proposed changes shortly. |
README.md
Outdated
@@ -46,6 +46,14 @@ If you don't have `rust` installed, we recommend installing it via [`rustup`](ht | |||
(If you *are* interested in having a binary release, then please let us know in the 'Discussions' area of this | |||
project or by filing a feature request in 'Issues'.) | |||
|
|||
[](https://repology.org/project/brush/versions) |
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 a fan of repology; would be great to include the badge. Could you move the badge up with the rest of the badges near the top of the README? That will fit in well.
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.
Sure! I have moved it with adding a blank line to fix mismatch of different badge height.
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.
This is the link for other styles of badges we can choose. https://repology.org/project/brush/badges
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.
Thanks for the info; the current badge looks fine for now.
Performance Benchmark Report
Code Coverage Report: Only Changed Files listed
Minimum allowed coverage is Test Summary: bash-completion test suite
|
Merged. Thanks for making this change and for pursuing |
Thank you for beginning this project! |
I have added the package definition in NixOS/nixpkgs#382795.
I hope this will make it easy to use by visitors without the build step. A binary cache of Nix might be an alternative solution for #69.