{"action":"created","issue":{"url":"https://api.github.com/repos/LSST/sg/issues/1","labels_url":"https://api.github.com/repos/LSST/sg/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/LSST/sg/issues/1/comments","events_url":"https://api.github.com/repos/LSST/sg/issues/1/events","html_url":"https://github.com/LSST/sg/pull/1","id":56596668,"number":1,"title":"Tickets/dm 215","user":{"login":"fritzm","id":1530638,"avatar_url":"https://avatars.githubusercontent.com/u/1530638?v=3","gravatar_id":"","url":"https://api.github.com/users/fritzm","html_url":"https://github.com/fritzm","followers_url":"https://api.github.com/users/fritzm/followers","following_url":"https://api.github.com/users/fritzm/following{/other_user}","gists_url":"https://api.github.com/users/fritzm/gists{/gist_id}","starred_url":"https://api.github.com/users/fritzm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fritzm/subscriptions","organizations_url":"https://api.github.com/users/fritzm/orgs","repos_url":"https://api.github.com/users/fritzm/repos","events_url":"https://api.github.com/users/fritzm/events{/privacy}","received_events_url":"https://api.github.com/users/fritzm/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-02-04T22:19:31Z","updated_at":"2015-02-06T06:52:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/LSST/sg/pulls/1","html_url":"https://github.com/LSST/sg/pull/1","diff_url":"https://github.com/LSST/sg/pull/1.diff","patch_url":"https://github.com/LSST/sg/pull/1.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/LSST/sg/issues/comments/73190448","html_url":"https://github.com/LSST/sg/pull/1#issuecomment-73190448","issue_url":"https://api.github.com/repos/LSST/sg/issues/1","id":73190448,"user":{"login":"TallJimbo","id":1330720,"avatar_url":"https://avatars.githubusercontent.com/u/1330720?v=3","gravatar_id":"","url":"https://api.github.com/users/TallJimbo","html_url":"https://github.com/TallJimbo","followers_url":"https://api.github.com/users/TallJimbo/followers","following_url":"https://api.github.com/users/TallJimbo/following{/other_user}","gists_url":"https://api.github.com/users/TallJimbo/gists{/gist_id}","starred_url":"https://api.github.com/users/TallJimbo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/TallJimbo/subscriptions","organizations_url":"https://api.github.com/users/TallJimbo/orgs","repos_url":"https://api.github.com/users/TallJimbo/repos","events_url":"https://api.github.com/users/TallJimbo/events{/privacy}","received_events_url":"https://api.github.com/users/TallJimbo/received_events","type":"User","site_admin":false},"created_at":"2015-02-06T06:52:30Z","updated_at":"2015-02-06T06:52:30Z","body":"A couple of general comments so far:\r\n - While we still have a lot of these lines in old code, I'm not a big fan of putting Doxygen `\\author` tags in files, as they're obsoleted as soon as someone else edits the file, and we should be using git to track that anyway.\r\n - To conform to our coding standards, your classes generally need a lot more whitespace - newlines between function declarations.  I think exceptions can be made (even if it's not in the standards) for groups of very closely-related functions (e.g. operator overloads) that each only occupy one line, but I think anything that merits its own Doxygen comment definitely merits a newline to separate it from its neighbors."}}