GitHub PullRequestReviewCommentEvent at 2014-01-28 18:28:28
Repository: melpa
Actor: purcell
Payload: {"comment":{"url":"https://api.github.com/repos/milkypostman/melpa/pulls/comments/9237910","id":9237910,"diff_hunk":"@@ -149,14 +149,18 @@ function for access to this function\")\n   \"In DIR (or `default-directory' if unset) run COMMAND with ARGS.\n Output is written to the current buffer.\"\n   (let* ((default-directory (or dir default-directory))\n+         (default-directory\n+           (if (file-directory-p default-directory)\n+               (concat (directory-file-name default-directory) \"/\")","path":"package-build.el","position":6,"original_position":6,"commit_id":"bf2d52b26b2008d16c232f08aac637fa297f267e","original_commit_id":"bf2d52b26b2008d16c232f08aac637fa297f267e","user":{"login":"purcell","id":5636,"avatar_url":"https://gravatar.com/avatar/04f3babefa4ba85e7904b40e6846e217?d=https%3A%2F%2Fidenticons.github.com%2F203cb085a5c2c3faf8e4f60131817256.png&r=x","gravatar_id":"04f3babefa4ba85e7904b40e6846e217","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},"body":"The double-let assignment shouldn't be necessary here..\r\n\r\n`file-name-as-directory` does the same as this `concat`, I believe, so is preferable.","created_at":"2014-01-28T18:28:28Z","updated_at":"2014-01-28T18:28:28Z","html_url":"https://github.com/milkypostman/melpa/pull/1432#discussion_r9237910","pull_request_url":"https://api.github.com/repos/milkypostman/melpa/pulls/1432","_links":{"self":{"href":"https://api.github.com/repos/milkypostman/melpa/pulls/comments/9237910"},"html":{"href":"https://github.com/milkypostman/melpa/pull/1432#discussion_r9237910"},"pull_request":{"href":"https://api.github.com/repos/milkypostman/melpa/pulls/1432"}}}}