{"action":"created","issue":{"url":"https://api.github.com/repos/milkypostman/melpa/issues/2791","labels_url":"https://api.github.com/repos/milkypostman/melpa/issues/2791/labels{/name}","comments_url":"https://api.github.com/repos/milkypostman/melpa/issues/2791/comments","events_url":"https://api.github.com/repos/milkypostman/melpa/issues/2791/events","html_url":"https://github.com/milkypostman/melpa/pull/2791","id":79015637,"number":2791,"title":"Add recipe for pyfmt","user":{"login":"aheaume","id":3447602,"avatar_url":"https://avatars.githubusercontent.com/u/3447602?v=3","gravatar_id":"","url":"https://api.github.com/users/aheaume","html_url":"https://github.com/aheaume","followers_url":"https://api.github.com/users/aheaume/followers","following_url":"https://api.github.com/users/aheaume/following{/other_user}","gists_url":"https://api.github.com/users/aheaume/gists{/gist_id}","starred_url":"https://api.github.com/users/aheaume/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aheaume/subscriptions","organizations_url":"https://api.github.com/users/aheaume/orgs","repos_url":"https://api.github.com/users/aheaume/repos","events_url":"https://api.github.com/users/aheaume/events{/privacy}","received_events_url":"https://api.github.com/users/aheaume/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-05-21T14:21:01Z","updated_at":"2015-05-21T19:15:07Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/milkypostman/melpa/pulls/2791","html_url":"https://github.com/milkypostman/melpa/pull/2791","diff_url":"https://github.com/milkypostman/melpa/pull/2791.diff","patch_url":"https://github.com/milkypostman/melpa/pull/2791.patch"},"body":"Hi!\r\n\r\nThis package is a simple interface to [pyfmt](https://github.com/Psycojoker/pyfmt) for formatting Python buffers.\r\n\r\nYou can find the repo here: [aheaume/pyfmt.el](https://github.com/aheaume/pyfmt.el). I'm the maintainer.\r\n\r\nThanks & have a nice day."},"comment":{"url":"https://api.github.com/repos/milkypostman/melpa/issues/comments/104392506","html_url":"https://github.com/milkypostman/melpa/pull/2791#issuecomment-104392506","issue_url":"https://api.github.com/repos/milkypostman/melpa/issues/2791","id":104392506,"user":{"login":"purcell","id":5636,"avatar_url":"https://avatars.githubusercontent.com/u/5636?v=3","gravatar_id":"","url":"https://api.github.com/users/purcell","html_url":"https://github.com/purcell","followers_url":"https://api.github.com/users/purcell/followers","following_url":"https://api.github.com/users/purcell/following{/other_user}","gists_url":"https://api.github.com/users/purcell/gists{/gist_id}","starred_url":"https://api.github.com/users/purcell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/purcell/subscriptions","organizations_url":"https://api.github.com/users/purcell/orgs","repos_url":"https://api.github.com/users/purcell/repos","events_url":"https://api.github.com/users/purcell/events{/privacy}","received_events_url":"https://api.github.com/users/purcell/received_events","type":"User","site_admin":false},"created_at":"2015-05-21T19:15:07Z","updated_at":"2015-05-21T19:15:07Z","body":"Thanks. You should add a `defgroup` to define the `pyfmt` customization group, and place that in the correct top-level group (`languages`, perhaps?).\r\n\r\nAlso, it would be better to call `error` when the command can't be found, so you can just say:\r\n\r\n```el\r\n(defun pyfmt ()\r\n  \"Format the current buffer according to the pyfmt tool.\"\r\n  (interactive)\r\n  (unless (executable-find pyfmt-command)\r\n    (error \"Can't find %s in the exec path\" pyfmt-command))\r\n  (call-process-region (point-min) (point-max) pyfmt-command t t nil (buffer-file-name)))\r\n```\r\n\r\n(Note that `error` and `message` both work like `format`.)\r\n\r\nIn fact, I think the check is redundant, because `call-process-region` should fail anyway if the command can't be found."}}