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
brotab: init at 0.0.5 #73777
brotab: init at 0.0.5 #73777
Conversation
220f380
to
4ff5b50
Compare
Thanks for the comments @jonringer ! |
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 repo seems to be most a 1 person show with no other repositories currently packaging this.
Why should ilt be added to nixpkgs?
Well first of all, the fact it's not famous enough so that other repos will include it doesn't mean the package is not useful. The least I can in order to support this software is to distribute it, starting with NixOS. Besides that, it's not just a script that one can |
What say you @jonringer ? |
328289a
to
df17bf6
Compare
df17bf6
to
052c103
Compare
Co-Authored-By: Jon <jonringer@users.noreply.github.com>
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.
diff LGTM
commits LGTM
has tests 👍
[3 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/73777
1 package built:
brotab
@GrahamcOfBorg build brotab |
This is actually a pretty cool tool! |
Thanks! 😄 |
brotab: init at 0.0.5 (cherry picked from commit 8cc8b3b)
Motivation for this change
Browser tab controller - https://github.com/balta2ar/brotab
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @