{"action":"created","issue":{"url":"https://api.github.com/repos/geoffmcl/test-sg/issues/5","repository_url":"https://api.github.com/repos/geoffmcl/test-sg","labels_url":"https://api.github.com/repos/geoffmcl/test-sg/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/geoffmcl/test-sg/issues/5/comments","events_url":"https://api.github.com/repos/geoffmcl/test-sg/issues/5/events","html_url":"https://github.com/geoffmcl/test-sg/pull/5","id":154928694,"number":5,"title":"I fixed my typo","user":{"login":"jasin","id":112683,"avatar_url":"https://avatars.githubusercontent.com/u/112683?v=3","gravatar_id":"","url":"https://api.github.com/users/jasin","html_url":"https://github.com/jasin","followers_url":"https://api.github.com/users/jasin/followers","following_url":"https://api.github.com/users/jasin/following{/other_user}","gists_url":"https://api.github.com/users/jasin/gists{/gist_id}","starred_url":"https://api.github.com/users/jasin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasin/subscriptions","organizations_url":"https://api.github.com/users/jasin/orgs","repos_url":"https://api.github.com/users/jasin/repos","events_url":"https://api.github.com/users/jasin/events{/privacy}","received_events_url":"https://api.github.com/users/jasin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-05-15T20:34:10Z","updated_at":"2016-05-15T21:34:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/geoffmcl/test-sg/pulls/5","html_url":"https://github.com/geoffmcl/test-sg/pull/5","diff_url":"https://github.com/geoffmcl/test-sg/pull/5.diff","patch_url":"https://github.com/geoffmcl/test-sg/pull/5.patch"},"body":"I removed _selectMSVC2.bat as well. Since we are now done with the testing."},"comment":{"url":"https://api.github.com/repos/geoffmcl/test-sg/issues/comments/219312034","html_url":"https://github.com/geoffmcl/test-sg/pull/5#issuecomment-219312034","issue_url":"https://api.github.com/repos/geoffmcl/test-sg/issues/5","id":219312034,"user":{"login":"geoffmcl","id":1333435,"avatar_url":"https://avatars.githubusercontent.com/u/1333435?v=3","gravatar_id":"","url":"https://api.github.com/users/geoffmcl","html_url":"https://github.com/geoffmcl","followers_url":"https://api.github.com/users/geoffmcl/followers","following_url":"https://api.github.com/users/geoffmcl/following{/other_user}","gists_url":"https://api.github.com/users/geoffmcl/gists{/gist_id}","starred_url":"https://api.github.com/users/geoffmcl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geoffmcl/subscriptions","organizations_url":"https://api.github.com/users/geoffmcl/orgs","repos_url":"https://api.github.com/users/geoffmcl/repos","events_url":"https://api.github.com/users/geoffmcl/events{/privacy}","received_events_url":"https://api.github.com/users/geoffmcl/received_events","type":"User","site_admin":false},"created_at":"2016-05-15T21:34:55Z","updated_at":"2016-05-15T21:34:55Z","body":"@jasin wow, I wish you could see what you presented, here, as a PR...\r\n\r\nFirst there are more than a dozen commits... **more** that 12...\r\n\r\nHow can I review all these?\r\n\r\nWhoh! They may **all** be right, correct, needed, but such a **complex** PR will for sure to fail a  review... I just do not have the time, or desire, to review **each** change, some of which, I can see,  are already in the `master`...\r\n\r\nI agree, there is quite a problem with `forks`... sometimes, it is too easy to modify it, usually for the better, and present a PR...\r\n\r\nBUT, and this is a big **but**, so long as the fork has been 100% rebased to the original... I tried earlier to give some advice on this... it is 100% important that $ git rebase upstream/master works 100% without problems...\r\n\r\nIn the past, when I have adjusted my fork to present a PR to the origin, but the origin has moved on, it can be difficult to bring the fork totally back into line...\r\n\r\nAnd as suggested, now, and then, it was necessary to delete my fork, and start again from the root..\r\n\r\nI have already advised, when I cloned your fork, and tried to rebase it to the original repo, I got **CONFLICTS** on the merge... and had **big** trouble trying to resolve these... read FAILED!\r\n\r\nNow @jasin I think you want to help, and really take the **extra** step of a PR... thank you for this.. but as stated above, the changes are too great to make an easy review possible, so it is **rejected**!\r\n\r\nAnother good idea, once you have your fork 100% back to this repo... is maybe good to represent a new idea in a branch...\r\n\r\nThere is a lot less review needed to `merge` a branch... switch to it, and give it a try...\r\n\r\nAs I say, loving the enthusiasm, but help me enjoy, make it simple, to use that...\r\n\r\n\r\n"}}