{"action":"created","comment":{"url":"https://api.github.com/repos/flycheck/flycheck/pulls/comments/41234382","id":41234382,"diff_hunk":"@@ -6872,6 +6873,22 @@ See URL `https://github.com/zaach/jsonlint'.\"\n      (and (buffer-file-name)\n           (string= (file-name-extension (buffer-file-name)) \"json\")))))\n \n+(flycheck-define-checker json-pythonjson\n+  \"A JSON syntax checker using Python json.tool module.\n+\n+See URL `https://docs.python.org/3.5/library/json.html#command-line-interface'.\"\n+  :command (\"python\" \"-m\" \"json.tool\" source-inplace null-device)","path":"flycheck.el","position":16,"original_position":16,"commit_id":"cc085f4aec7bdef011a8fdb622e28098e02979dc","original_commit_id":"cc085f4aec7bdef011a8fdb622e28098e02979dc","user":{"login":"lunaryorn","id":224922,"avatar_url":"https://avatars.githubusercontent.com/u/224922?v=3","gravatar_id":"","url":"https://api.github.com/users/lunaryorn","html_url":"https://github.com/lunaryorn","followers_url":"https://api.github.com/users/lunaryorn/followers","following_url":"https://api.github.com/users/lunaryorn/following{/other_user}","gists_url":"https://api.github.com/users/lunaryorn/gists{/gist_id}","starred_url":"https://api.github.com/users/lunaryorn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lunaryorn/subscriptions","organizations_url":"https://api.github.com/users/lunaryorn/orgs","repos_url":"https://api.github.com/users/lunaryorn/repos","events_url":"https://api.github.com/users/lunaryorn/events{/privacy}","received_events_url":"https://api.github.com/users/lunaryorn/received_events","type":"User","site_admin":false},"body":"Why do you use `source-inplace` here?  Is that really necessary?  Can't we use `source` instead?\r\n\r\nI presume `null-device` is for the (pretty-printed?) output?   If so, please add a little comment that explains it, so that I don't need to look up documentation if I need to understand this syntax checker in half a year from now or so :relaxed: ","created_at":"2015-10-06T07:57:57Z","updated_at":"2015-10-06T07:57:57Z","html_url":"https://github.com/flycheck/flycheck/pull/758#discussion_r41234382","pull_request_url":"https://api.github.com/repos/flycheck/flycheck/pulls/758","_links":{"self":{"href":"https://api.github.com/repos/flycheck/flycheck/pulls/comments/41234382"},"html":{"href":"https://github.com/flycheck/flycheck/pull/758#discussion_r41234382"},"pull_request":{"href":"https://api.github.com/repos/flycheck/flycheck/pulls/758"}}},"pull_request":{"url":"https://api.github.com/repos/flycheck/flycheck/pulls/758","id":46860580,"html_url":"https://github.com/flycheck/flycheck/pull/758","diff_url":"https://github.com/flycheck/flycheck/pull/758.diff","patch_url":"https://github.com/flycheck/flycheck/pull/758.patch","issue_url":"https://api.github.com/repos/flycheck/flycheck/issues/758","number":758,"state":"open","locked":false,"title":"Add json checking via python json.tool module","user":{"login":"marsam","id":65531,"avatar_url":"https://avatars.githubusercontent.com/u/65531?v=3","gravatar_id":"","url":"https://api.github.com/users/marsam","html_url":"https://github.com/marsam","followers_url":"https://api.github.com/users/marsam/followers","following_url":"https://api.github.com/users/marsam/following{/other_user}","gists_url":"https://api.github.com/users/marsam/gists{/gist_id}","starred_url":"https://api.github.com/users/marsam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marsam/subscriptions","organizations_url":"https://api.github.com/users/marsam/orgs","repos_url":"https://api.github.com/users/marsam/repos","events_url":"https://api.github.com/users/marsam/events{/privacy}","received_events_url":"https://api.github.com/users/marsam/received_events","type":"User","site_admin":false},"body":"Python `json.tool` module  is available in both python2 and python3.","created_at":"2015-10-06T04:02:40Z","updated_at":"2015-10-06T07:57:58Z","closed_at":null,"merged_at":null,"merge_commit_sha":"bd34ff6c7a96f797abafb55c85d53173c44438ac","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/flycheck/flycheck/pulls/758/commits","review_comments_url":"https://api.github.com/repos/flycheck/flycheck/pulls/758/comments","review_comment_url":"https://api.github.com/repos/flycheck/flycheck/pulls/comments{/number}","comments_url":"https://api.github.com/repos/flycheck/flycheck/issues/758/comments","statuses_url":"https://api.github.com/repos/flycheck/flycheck/statuses/cc085f4aec7bdef011a8fdb622e28098e02979dc","head":{"label":"marsam:master","ref":"master","sha":"cc085f4aec7bdef011a8fdb622e28098e02979dc","user":{"login":"marsam","id":65531,"avatar_url":"https://avatars.githubusercontent.com/u/65531?v=3","gravatar_id":"","url":"https://api.github.com/users/marsam","html_url":"https://github.com/marsam","followers_url":"https://api.github.com/users/marsam/followers","following_url":"https://api.github.com/users/marsam/following{/other_user}","gists_url":"https://api.github.com/users/marsam/gists{/gist_id}","starred_url":"https://api.github.com/users/marsam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marsam/subscriptions","organizations_url":"https://api.github.com/users/marsam/orgs","repos_url":"https://api.github.com/users/marsam/repos","events_url":"https://api.github.com/users/marsam/events{/privacy}","received_events_url":"https://api.github.com/users/marsam/received_events","type":"User","site_admin":false},"repo":{"id":43728842,"name":"flycheck","full_name":"marsam/flycheck","owner":{"login":"marsam","id":65531,"avatar_url":"https://avatars.githubusercontent.com/u/65531?v=3","gravatar_id":"","url":"https://api.github.com/users/marsam","html_url":"https://github.com/marsam","followers_url":"https://api.github.com/users/marsam/followers","following_url":"https://api.github.com/users/marsam/following{/other_user}","gists_url":"https://api.github.com/users/marsam/gists{/gist_id}","starred_url":"https://api.github.com/users/marsam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marsam/subscriptions","organizations_url":"https://api.github.com/users/marsam/orgs","repos_url":"https://api.github.com/users/marsam/repos","events_url":"https://api.github.com/users/marsam/events{/privacy}","received_events_url":"https://api.github.com/users/marsam/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/marsam/flycheck","description":"Modern on the fly syntax checking for GNU Emacs","fork":true,"url":"https://api.github.com/repos/marsam/flycheck","forks_url":"https://api.github.com/repos/marsam/flycheck/forks","keys_url":"https://api.github.com/repos/marsam/flycheck/keys{/key_id}","collaborators_url":"https://api.github.com/repos/marsam/flycheck/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/marsam/flycheck/teams","hooks_url":"https://api.github.com/repos/marsam/flycheck/hooks","issue_events_url":"https://api.github.com/repos/marsam/flycheck/issues/events{/number}","events_url":"https://api.github.com/repos/marsam/flycheck/events","assignees_url":"https://api.github.com/repos/marsam/flycheck/assignees{/user}","branches_url":"https://api.github.com/repos/marsam/flycheck/branches{/branch}","tags_url":"https://api.github.com/repos/marsam/flycheck/tags","blobs_url":"https://api.github.com/repos/marsam/flycheck/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/marsam/flycheck/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/marsam/flycheck/git/refs{/sha}","trees_url":"https://api.github.com/repos/marsam/flycheck/git/trees{/sha}","statuses_url":"https://api.github.com/repos/marsam/flycheck/statuses/{sha}","languages_url":"https://api.github.com/repos/marsam/flycheck/languages","stargazers_url":"https://api.github.com/repos/marsam/flycheck/stargazers","contributors_url":"https://api.github.com/repos/marsam/flycheck/contributors","subscribers_url":"https://api.github.com/repos/marsam/flycheck/subscribers","subscription_url":"https://api.github.com/repos/marsam/flycheck/subscription","commits_url":"https://api.github.com/repos/marsam/flycheck/commits{/sha}","git_commits_url":"https://api.github.com/repos/marsam/flycheck/git/commits{/sha}","comments_url":"https://api.github.com/repos/marsam/flycheck/comments{/number}","issue_comment_url":"https://api.github.com/repos/marsam/flycheck/issues/comments{/number}","contents_url":"https://api.github.com/repos/marsam/flycheck/contents/{+path}","compare_url":"https://api.github.com/repos/marsam/flycheck/compare/{base}...{head}","merges_url":"https://api.github.com/repos/marsam/flycheck/merges","archive_url":"https://api.github.com/repos/marsam/flycheck/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/marsam/flycheck/downloads","issues_url":"https://api.github.com/repos/marsam/flycheck/issues{/number}","pulls_url":"https://api.github.com/repos/marsam/flycheck/pulls{/number}","milestones_url":"https://api.github.com/repos/marsam/flycheck/milestones{/number}","notifications_url":"https://api.github.com/repos/marsam/flycheck/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/marsam/flycheck/labels{/name}","releases_url":"https://api.github.com/repos/marsam/flycheck/releases{/id}","created_at":"2015-10-06T03:57:00Z","updated_at":"2015-10-06T03:57:03Z","pushed_at":"2015-10-06T04:00:27Z","git_url":"git://github.com/marsam/flycheck.git","ssh_url":"git@github.com:marsam/flycheck.git","clone_url":"https://github.com/marsam/flycheck.git","svn_url":"https://github.com/marsam/flycheck","homepage":"http://www.flycheck.org","size":21141,"stargazers_count":0,"watchers_count":0,"language":"Emacs Lisp","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"flycheck:master","ref":"master","sha":"53acb7436fd8c350952e338b4a04f6f4d77d5788","user":{"login":"flycheck","id":5254142,"avatar_url":"https://avatars.githubusercontent.com/u/5254142?v=3","gravatar_id":"","url":"https://api.github.com/users/flycheck","html_url":"https://github.com/flycheck","followers_url":"https://api.github.com/users/flycheck/followers","following_url":"https://api.github.com/users/flycheck/following{/other_user}","gists_url":"https://api.github.com/users/flycheck/gists{/gist_id}","starred_url":"https://api.github.com/users/flycheck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/flycheck/subscriptions","organizations_url":"https://api.github.com/users/flycheck/orgs","repos_url":"https://api.github.com/users/flycheck/repos","events_url":"https://api.github.com/users/flycheck/events{/privacy}","received_events_url":"https://api.github.com/users/flycheck/received_events","type":"Organization","site_admin":false},"repo":{"id":6158882,"name":"flycheck","full_name":"flycheck/flycheck","owner":{"login":"flycheck","id":5254142,"avatar_url":"https://avatars.githubusercontent.com/u/5254142?v=3","gravatar_id":"","url":"https://api.github.com/users/flycheck","html_url":"https://github.com/flycheck","followers_url":"https://api.github.com/users/flycheck/followers","following_url":"https://api.github.com/users/flycheck/following{/other_user}","gists_url":"https://api.github.com/users/flycheck/gists{/gist_id}","starred_url":"https://api.github.com/users/flycheck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/flycheck/subscriptions","organizations_url":"https://api.github.com/users/flycheck/orgs","repos_url":"https://api.github.com/users/flycheck/repos","events_url":"https://api.github.com/users/flycheck/events{/privacy}","received_events_url":"https://api.github.com/users/flycheck/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/flycheck/flycheck","description":"Modern on the fly syntax checking for GNU Emacs","fork":false,"url":"https://api.github.com/repos/flycheck/flycheck","forks_url":"https://api.github.com/repos/flycheck/flycheck/forks","keys_url":"https://api.github.com/repos/flycheck/flycheck/keys{/key_id}","collaborators_url":"https://api.github.com/repos/flycheck/flycheck/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/flycheck/flycheck/teams","hooks_url":"https://api.github.com/repos/flycheck/flycheck/hooks","issue_events_url":"https://api.github.com/repos/flycheck/flycheck/issues/events{/number}","events_url":"https://api.github.com/repos/flycheck/flycheck/events","assignees_url":"https://api.github.com/repos/flycheck/flycheck/assignees{/user}","branches_url":"https://api.github.com/repos/flycheck/flycheck/branches{/branch}","tags_url":"https://api.github.com/repos/flycheck/flycheck/tags","blobs_url":"https://api.github.com/repos/flycheck/flycheck/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/flycheck/flycheck/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/flycheck/flycheck/git/refs{/sha}","trees_url":"https://api.github.com/repos/flycheck/flycheck/git/trees{/sha}","statuses_url":"https://api.github.com/repos/flycheck/flycheck/statuses/{sha}","languages_url":"https://api.github.com/repos/flycheck/flycheck/languages","stargazers_url":"https://api.github.com/repos/flycheck/flycheck/stargazers","contributors_url":"https://api.github.com/repos/flycheck/flycheck/contributors","subscribers_url":"https://api.github.com/repos/flycheck/flycheck/subscribers","subscription_url":"https://api.github.com/repos/flycheck/flycheck/subscription","commits_url":"https://api.github.com/repos/flycheck/flycheck/commits{/sha}","git_commits_url":"https://api.github.com/repos/flycheck/flycheck/git/commits{/sha}","comments_url":"https://api.github.com/repos/flycheck/flycheck/comments{/number}","issue_comment_url":"https://api.github.com/repos/flycheck/flycheck/issues/comments{/number}","contents_url":"https://api.github.com/repos/flycheck/flycheck/contents/{+path}","compare_url":"https://api.github.com/repos/flycheck/flycheck/compare/{base}...{head}","merges_url":"https://api.github.com/repos/flycheck/flycheck/merges","archive_url":"https://api.github.com/repos/flycheck/flycheck/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/flycheck/flycheck/downloads","issues_url":"https://api.github.com/repos/flycheck/flycheck/issues{/number}","pulls_url":"https://api.github.com/repos/flycheck/flycheck/pulls{/number}","milestones_url":"https://api.github.com/repos/flycheck/flycheck/milestones{/number}","notifications_url":"https://api.github.com/repos/flycheck/flycheck/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/flycheck/flycheck/labels{/name}","releases_url":"https://api.github.com/repos/flycheck/flycheck/releases{/id}","created_at":"2012-10-10T14:45:13Z","updated_at":"2015-10-06T02:23:47Z","pushed_at":"2015-10-06T04:02:41Z","git_url":"git://github.com/flycheck/flycheck.git","ssh_url":"git@github.com:flycheck/flycheck.git","clone_url":"https://github.com/flycheck/flycheck.git","svn_url":"https://github.com/flycheck/flycheck","homepage":"http://www.flycheck.org","size":21141,"stargazers_count":810,"watchers_count":810,"language":"Emacs Lisp","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":155,"mirror_url":null,"open_issues_count":21,"forks":155,"open_issues":21,"watchers":810,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/flycheck/flycheck/pulls/758"},"html":{"href":"https://github.com/flycheck/flycheck/pull/758"},"issue":{"href":"https://api.github.com/repos/flycheck/flycheck/issues/758"},"comments":{"href":"https://api.github.com/repos/flycheck/flycheck/issues/758/comments"},"review_comments":{"href":"https://api.github.com/repos/flycheck/flycheck/pulls/758/comments"},"review_comment":{"href":"https://api.github.com/repos/flycheck/flycheck/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/flycheck/flycheck/pulls/758/commits"},"statuses":{"href":"https://api.github.com/repos/flycheck/flycheck/statuses/cc085f4aec7bdef011a8fdb622e28098e02979dc"}}}}