{"comment":{"url":"https://api.github.com/repos/valoriedodge/Pomodoro-Project/comments/19064886","html_url":"https://github.com/valoriedodge/Pomodoro-Project/commit/31e26210ad1cc3c7bd940b0e34f83970906d482d#commitcomment-19064886","id":19064886,"user":{"login":"rdnewman","id":5544943,"avatar_url":"https://avatars.githubusercontent.com/u/5544943?v=3","gravatar_id":"","url":"https://api.github.com/users/rdnewman","html_url":"https://github.com/rdnewman","followers_url":"https://api.github.com/users/rdnewman/followers","following_url":"https://api.github.com/users/rdnewman/following{/other_user}","gists_url":"https://api.github.com/users/rdnewman/gists{/gist_id}","starred_url":"https://api.github.com/users/rdnewman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rdnewman/subscriptions","organizations_url":"https://api.github.com/users/rdnewman/orgs","repos_url":"https://api.github.com/users/rdnewman/repos","events_url":"https://api.github.com/users/rdnewman/events{/privacy}","received_events_url":"https://api.github.com/users/rdnewman/received_events","type":"User","site_admin":false},"position":261,"line":4,"path":"app/scripts/services/Timer.js","commit_id":"31e26210ad1cc3c7bd940b0e34f83970906d482d","created_at":"2016-09-17T14:38:57Z","updated_at":"2016-09-17T14:38:57Z","body":"You should only use snake_case when naming local variables.   PascalCase (camelcase with initial letter capitalized) should be used for classes and constructors.  camelCase (PascalCase with initial letter lowercase) for objects, etc.\r\n\r\nNote that in this function, your local \"Timer\" and the function name \"Timer\" is a naming collision.   I'd probably prefer that your line 3 just go with \"timer\" if you intend that to be separate object from the function.\r\n\r\nAlso see https://github.com/airbnb/javascript#variables and https://github.com/mgechev/angularjs-style-guide for other style advice. "}}
{"actor":{"display_login":"rdnewman"}}