Skip to content
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

Removed trailing comma from swaybar click event json #1441

Merged
merged 1 commit into from
Oct 28, 2017
Merged

Removed trailing comma from swaybar click event json #1441

merged 1 commit into from
Oct 28, 2017

Conversation

ptMuta
Copy link
Contributor

@ptMuta ptMuta commented Oct 28, 2017

Trailing comma was violating JSON validity and caused issues with deserialization in different status_command implementations (eg. i3status-rust)

@ddevault
Copy link
Contributor

Please associate your committer email with your GitHub account.

@ddevault
Copy link
Contributor

Hmm, that's not quite right.

@ptMuta
Copy link
Contributor Author

ptMuta commented Oct 28, 2017

There we go... First time I had to correct commiting with a wrong profile

@ddevault
Copy link
Contributor

Thanks!

@ddevault ddevault merged commit ccf7e5a into swaywm:master Oct 28, 2017
ddevault added a commit that referenced this pull request Oct 30, 2017
Removed trailing comma from swaybar click event json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants