{"action":"opened","number":1,"pull_request":{"review_comments":0,"created_at":"2011-12-10T12:15:33Z","state":"open","updated_at":"2011-12-10T12:15:33Z","changed_files":15,"_links":{"review_comments":{"href":"https://api.github.com/repos/FuzzAU/MinesweeperBot/pulls/1/comments"},"comments":{"href":"https://api.github.com/repos/FuzzAU/MinesweeperBot/issues/1/comments"},"html":{"href":"https://github.com/FuzzAU/MinesweeperBot/pull/1"},"self":{"href":"https://api.github.com/repos/FuzzAU/MinesweeperBot/pulls/1"}},"patch_url":"https://github.com/FuzzAU/MinesweeperBot/pull/1.patch","issue_url":"https://github.com/FuzzAU/MinesweeperBot/issues/1","comments":0,"user":{"gravatar_id":"0060becade3fe5d19645ad316beea2f3","id":473297,"url":"https://api.github.com/users/AndrewWalker","avatar_url":"https://secure.gravatar.com/avatar/0060becade3fe5d19645ad316beea2f3?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"AndrewWalker"},"merged":false,"body":"I've added a Makefile, added a lint target, tweaked the tests, added a few new tests (and found a subtle bug). Also, IMO you can get rid of <root>/src/, just put everything in <root>/src in <root>.\r\n\r\nFrom a convention standpoint, python files should probably in lower case, no underscore or hyphens. See http://www.python.org/dev/peps/pep-0008/ for more info but in short:\r\n\r\nclasses should be LikeThis\r\nfunctions should be like_this\r\nconstants should be LIKE_THIS\r\nvariables should be like_this\r\n\r\nConsidering this is your first python project, this is looking quite impressive","deletions":95,"mergeable":null,"commits":23,"merged_by":null,"id":580257,"additions":198,"base":{"repo":{"pushed_at":"2011-12-10T09:50:17Z","created_at":"2011-12-07T09:15:43Z","updated_at":"2011-12-10T09:54:27Z","private":false,"clone_url":"https://github.com/FuzzAU/MinesweeperBot.git","name":"MinesweeperBot","watchers":2,"svn_url":"https://svn.github.com/FuzzAU/MinesweeperBot","master_branch":null,"forks":2,"language":null,"owner":{"gravatar_id":"a8a8c85bcc81664585f03bb5c247e00a","id":1161763,"url":"https://api.github.com/users/FuzzAU","avatar_url":"https://secure.gravatar.com/avatar/a8a8c85bcc81664585f03bb5c247e00a?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"FuzzAU"},"id":2931426,"homepage":"","description":"My attempt to make minesweeper, an AI to play it, and learn python","git_url":"git://github.com/FuzzAU/MinesweeperBot.git","html_url":"https://github.com/FuzzAU/MinesweeperBot","open_issues":1,"size":108,"url":"https://api.github.com/repos/FuzzAU/MinesweeperBot","fork":false,"ssh_url":"git@github.com:FuzzAU/MinesweeperBot.git"},"user":{"gravatar_id":"a8a8c85bcc81664585f03bb5c247e00a","id":1161763,"url":"https://api.github.com/users/FuzzAU","avatar_url":"https://secure.gravatar.com/avatar/a8a8c85bcc81664585f03bb5c247e00a?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"FuzzAU"},"sha":"fa29a56d869a4fd577097a61518517952221929f","ref":"master","label":"FuzzAU:master"},"number":1,"head":{"repo":{"pushed_at":"2011-12-10T12:07:52Z","created_at":"2011-12-10T09:54:27Z","updated_at":"2011-12-10T12:07:55Z","private":false,"clone_url":"https://github.com/AndrewWalker/MinesweeperBot.git","name":"MinesweeperBot","watchers":1,"svn_url":"https://svn.github.com/AndrewWalker/MinesweeperBot","master_branch":null,"forks":0,"language":"Python","owner":{"gravatar_id":"0060becade3fe5d19645ad316beea2f3","id":473297,"url":"https://api.github.com/users/AndrewWalker","avatar_url":"https://secure.gravatar.com/avatar/0060becade3fe5d19645ad316beea2f3?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"AndrewWalker"},"id":2952790,"homepage":"","description":"My attempt to make minesweeper, an AI to play it, and learn python","git_url":"git://github.com/AndrewWalker/MinesweeperBot.git","html_url":"https://github.com/AndrewWalker/MinesweeperBot","open_issues":0,"size":124,"url":"https://api.github.com/repos/AndrewWalker/MinesweeperBot","fork":true,"ssh_url":"git@github.com:AndrewWalker/MinesweeperBot.git"},"user":{"gravatar_id":"0060becade3fe5d19645ad316beea2f3","id":473297,"url":"https://api.github.com/users/AndrewWalker","avatar_url":"https://secure.gravatar.com/avatar/0060becade3fe5d19645ad316beea2f3?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"AndrewWalker"},"sha":"d441a8e5e9e67dd57c7d6427518f29c0fb048ae2","ref":"master","label":"AndrewWalker:master"},"title":"Andrew's Mainly lint/pep8 refactorings","html_url":"https://github.com/FuzzAU/MinesweeperBot/pull/1","closed_at":null,"url":"https://api.github.com/repos/FuzzAU/MinesweeperBot/pulls/1","diff_url":"https://github.com/FuzzAU/MinesweeperBot/pull/1.diff","merged_at":null}}