Skip to content

compton: fix syntax error #296

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

Merged
merged 1 commit into from
Jun 27, 2018
Merged

Conversation

dermetfan
Copy link
Collaborator

builtins.toString returns the empty string or 1 for boolean values, so toJSON is required instead.

In other places this was done correctly so this just seems to be a missed case.

@rycee rycee merged commit 0d3f9ba into nix-community:master Jun 27, 2018
@rycee
Copy link
Member

rycee commented Jun 27, 2018

Looking good. Thanks! 👍

@dermetfan dermetfan deleted the fix/compton-syntax branch June 27, 2018 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants