-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
bartender: add passthru.updateScript
, format with nixfmt-rfc-style
, use lib.platforms.darwin
, 5.2.3 -> 5.2.7
#363633
Conversation
0963e77
to
283f492
Compare
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.
Otherwise LGTM
d029c43
to
2e3ee0f
Compare
2e3ee0f
to
9d241e0
Compare
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.
Feel free to ping in a couple days if there are no other reviews
nixpkgs-review
result
Generated using nixpkgs-review
.
Command: nixpkgs-review pr 363633
aarch64-darwin
✅ 1 package built:
- bartender
|
@DimitarNestorov merge not permitted (#305350): |
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.