-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
polybar: use recursive config type #2235
Conversation
modules/services/polybar.nix
Outdated
@@ -112,7 +112,9 @@ in { | |||
}; | |||
|
|||
settings = mkOption { | |||
type = types.attrsOf types.attrs; | |||
type = with types; | |||
let ty = oneOf [ bool int str (listOf ty) (attrsOf ty) ]; |
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.
Shouldn't it also include floats?
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.
yup, thanks!
Allow merging of lists and attrsets
b30d6f3
to
c48f635
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.
Thanks.
Allow merging of lists and attrsets
Description
Allow merging of lists and attrsets in polybar configs.
(I believe I need to set a description or something to fix recursive types in the docs, what do I need to do again?)
Checklist
Change is backwards compatible.
Code formatted with
./format
.Code tested through
nix-shell --pure tests -A run.all
.Test cases updated/added. See example.
Commit messages are formatted like
See CONTRIBUTING for more information and recent commit messages for examples.
If this PR adds a new module
Added myself as module maintainer. See example.
Added myself and the module files to
.github/CODEOWNERS
.