{"pull_request":{"state":"open","deletions":7,"merged_by":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/cc8ce94e3a3339f0454d5e659e4aba8f?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"cc8ce94e3a3339f0454d5e659e4aba8f","url":"https://api.github.com/users/beanz","login":"beanz","id":79058},"updated_at":"2011-11-17T21:54:49Z","closed_at":null,"comments":0,"base":{"ref":"master","user":{"avatar_url":"https://secure.gravatar.com/avatar/7379345fed54f2447537068ed1c2e440?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"7379345fed54f2447537068ed1c2e440","url":"https://api.github.com/users/antirez","login":"antirez","id":65632},"repo":{"name":"lamernews","pushed_at":"2011-11-04T18:54:27Z","master_branch":null,"fork":false,"updated_at":"2011-11-17T13:03:19Z","clone_url":"https://github.com/antirez/lamernews.git","homepage":"http://lamernews.com","open_issues":66,"language":"Ruby","svn_url":"https://svn.github.com/antirez/lamernews","private":false,"forks":79,"url":"https://api.github.com/repos/antirez/lamernews","owner":{"avatar_url":"https://secure.gravatar.com/avatar/7379345fed54f2447537068ed1c2e440?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"7379345fed54f2447537068ed1c2e440","url":"https://api.github.com/users/antirez","login":"antirez","id":65632},"html_url":"https://github.com/antirez/lamernews","description":"Lamer News -- an HN style social news site written in Ruby/Sinatra/Redis/JQuery","size":776,"id":2581086,"watchers":530,"git_url":"git://github.com/antirez/lamernews.git","created_at":"2011-10-15T09:26:51Z","ssh_url":"git@github.com:antirez/lamernews.git"},"label":"antirez:master","sha":"89cde3acc13febff538ee225b66f6f2900e9dea6"},"diff_url":"https://github.com/antirez/lamernews/pull/113.diff","commits":1,"head":{"ref":"redundant-user-lookup","user":{"avatar_url":"https://secure.gravatar.com/avatar/cc8ce94e3a3339f0454d5e659e4aba8f?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"cc8ce94e3a3339f0454d5e659e4aba8f","url":"https://api.github.com/users/beanz","login":"beanz","id":79058},"repo":{"name":"lamernews","pushed_at":"2011-11-17T21:53:04Z","master_branch":null,"fork":true,"updated_at":"2011-11-17T21:53:05Z","clone_url":"https://github.com/beanz/lamernews.git","homepage":"http://lamernews.com","open_issues":0,"language":"Ruby","svn_url":"https://svn.github.com/beanz/lamernews","private":false,"forks":0,"url":"https://api.github.com/repos/beanz/lamernews","owner":{"avatar_url":"https://secure.gravatar.com/avatar/cc8ce94e3a3339f0454d5e659e4aba8f?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"cc8ce94e3a3339f0454d5e659e4aba8f","url":"https://api.github.com/users/beanz","login":"beanz","id":79058},"html_url":"https://github.com/beanz/lamernews","description":"Lamer News -- an HN style social news site written in Ruby/Sinatra/Redis/JQuery","size":148,"id":2651366,"watchers":1,"git_url":"git://github.com/beanz/lamernews.git","created_at":"2011-10-26T14:25:33Z","ssh_url":"git@github.com:beanz/lamernews.git"},"label":"beanz:redundant-user-lookup","sha":"ef9cd630ac13fef5a84f425749f27ad883300d0f"},"additions":5,"url":"https://api.github.com/repos/antirez/lamernews/pulls/113","merged":false,"body":"Both calls to update_auth_token had argument '$user[\"id\"]' and the\r\nfirst thing update_auth_token does is looks up the user record.\r\nChange update_auth_token api to take the user record instead.\r\n","merged_at":null,"html_url":"https://github.com/antirez/lamernews/pull/113","review_comments":0,"_links":{"comments":{"href":"https://api.github.com/repos/antirez/lamernews/issues/113/comments"},"html":{"href":"https://github.com/antirez/lamernews/pull/113"},"review_comments":{"href":"https://api.github.com/repos/antirez/lamernews/pulls/113/comments"},"self":{"href":"https://api.github.com/repos/antirez/lamernews/pulls/113"}},"issue_url":"https://github.com/antirez/lamernews/issues/113","title":"Avoid redundant user lookup but refactoring update_auth_token api.","number":113,"mergeable":null,"id":506253,"patch_url":"https://github.com/antirez/lamernews/pull/113.patch","changed_files":1,"created_at":"2011-11-17T21:54:49Z"},"action":"opened","number":113}