{"action":"created","issue":{"url":"https://api.github.com/repos/Goles/Battery/issues/22","labels_url":"https://api.github.com/repos/Goles/Battery/issues/22/labels{/name}","comments_url":"https://api.github.com/repos/Goles/Battery/issues/22/comments","events_url":"https://api.github.com/repos/Goles/Battery/issues/22/events","html_url":"https://github.com/Goles/Battery/pull/22","id":100520772,"number":22,"title":"Linux support merge","user":{"login":"minimenchmuncher","id":3420931,"avatar_url":"https://avatars.githubusercontent.com/u/3420931?v=3","gravatar_id":"","url":"https://api.github.com/users/minimenchmuncher","html_url":"https://github.com/minimenchmuncher","followers_url":"https://api.github.com/users/minimenchmuncher/followers","following_url":"https://api.github.com/users/minimenchmuncher/following{/other_user}","gists_url":"https://api.github.com/users/minimenchmuncher/gists{/gist_id}","starred_url":"https://api.github.com/users/minimenchmuncher/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/minimenchmuncher/subscriptions","organizations_url":"https://api.github.com/users/minimenchmuncher/orgs","repos_url":"https://api.github.com/users/minimenchmuncher/repos","events_url":"https://api.github.com/users/minimenchmuncher/events{/privacy}","received_events_url":"https://api.github.com/users/minimenchmuncher/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-08-12T11:16:28Z","updated_at":"2015-09-19T20:28:57Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Goles/Battery/pulls/22","html_url":"https://github.com/Goles/Battery/pull/22","diff_url":"https://github.com/Goles/Battery/pull/22.diff","patch_url":"https://github.com/Goles/Battery/pull/22.patch"},"body":"There were a couple of things here:\r\n\r\n* Added switch/case for different OSes based on `$(uname -s)`\r\n* Added a case for Linux\r\n* Removed `battery_info()` and `battery_external_connected` functions, moved their functionality into `battery_charge()`\r\n* checked whether `pmset` was present on the computer"},"comment":{"url":"https://api.github.com/repos/Goles/Battery/issues/comments/141703313","html_url":"https://github.com/Goles/Battery/pull/22#issuecomment-141703313","issue_url":"https://api.github.com/repos/Goles/Battery/issues/22","id":141703313,"user":{"login":"Goles","id":181342,"avatar_url":"https://avatars.githubusercontent.com/u/181342?v=3","gravatar_id":"","url":"https://api.github.com/users/Goles","html_url":"https://github.com/Goles","followers_url":"https://api.github.com/users/Goles/followers","following_url":"https://api.github.com/users/Goles/following{/other_user}","gists_url":"https://api.github.com/users/Goles/gists{/gist_id}","starred_url":"https://api.github.com/users/Goles/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Goles/subscriptions","organizations_url":"https://api.github.com/users/Goles/orgs","repos_url":"https://api.github.com/users/Goles/repos","events_url":"https://api.github.com/users/Goles/events{/privacy}","received_events_url":"https://api.github.com/users/Goles/received_events","type":"User","site_admin":false},"created_at":"2015-09-19T20:28:57Z","updated_at":"2015-09-19T20:28:57Z","body":"That's fantastic.\n\nWill take a deeper look today in the afternoon :)\n\nCheers!\nNico\n\nSent from my iPhone\n\n> On Sep 19, 2015, at 12:43 PM, Ezra Tucker <notifications@github.com> wrote:\n> \n> Hi @Goles, yes, this is safe to merge. I did one last test both on my mac and linux VM, also noted that my vm didn't have spark (which I realized might actually happen), so I added in one last little change to test for spark (using hash), and if it is there it will be used; if not, it reverts back to using ascii, and doesn't raise any errors or warnings.\n> \n> —\n> Reply to this email directly or view it on GitHub.\n> \n"}}