{"action":"created","issue":{"url":"https://api.github.com/repos/sfbrigade/brigadehub/issues/328","repository_url":"https://api.github.com/repos/sfbrigade/brigadehub","labels_url":"https://api.github.com/repos/sfbrigade/brigadehub/issues/328/labels{/name}","comments_url":"https://api.github.com/repos/sfbrigade/brigadehub/issues/328/comments","events_url":"https://api.github.com/repos/sfbrigade/brigadehub/issues/328/events","html_url":"https://github.com/sfbrigade/brigadehub/pull/328","id":158045278,"number":328,"title":"feat(metrics): fix codeclimate configuration","user":{"login":"dblandin","id":123595,"avatar_url":"https://avatars.githubusercontent.com/u/123595?v=3","gravatar_id":"","url":"https://api.github.com/users/dblandin","html_url":"https://github.com/dblandin","followers_url":"https://api.github.com/users/dblandin/followers","following_url":"https://api.github.com/users/dblandin/following{/other_user}","gists_url":"https://api.github.com/users/dblandin/gists{/gist_id}","starred_url":"https://api.github.com/users/dblandin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dblandin/subscriptions","organizations_url":"https://api.github.com/users/dblandin/orgs","repos_url":"https://api.github.com/users/dblandin/repos","events_url":"https://api.github.com/users/dblandin/events{/privacy}","received_events_url":"https://api.github.com/users/dblandin/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/sfbrigade/brigadehub/labels/broken%20Travis%20build","name":"broken Travis build","color":"eeeeee"},{"url":"https://api.github.com/repos/sfbrigade/brigadehub/labels/needs%20review","name":"needs review","color":"eb6420"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2016-06-02T02:10:15Z","updated_at":"2016-06-04T20:37:01Z","closed_at":"2016-06-04T18:12:27Z","pull_request":{"url":"https://api.github.com/repos/sfbrigade/brigadehub/pulls/328","html_url":"https://github.com/sfbrigade/brigadehub/pull/328","diff_url":"https://github.com/sfbrigade/brigadehub/pull/328.diff","patch_url":"https://github.com/sfbrigade/brigadehub/pull/328.patch"},"body":"### Description\r\n\r\nRemove unanalyzed file extensions from CC exclude_paths\r\n\r\n`*` at the beginning of a value produces invalid YAML. Normally the fix would\r\nbe to wrap these values in quotes, but in this case I think it makes more sense\r\nto remove the lines as these file extensions aren't analyzed by any enabled\r\nengine.\r\n\r\nAdd `npm-shrinkwrap.json` file for CC nodesecurity engine\r\n\r\nSince the Code Climate nodesecurity engine runs its underlying tool in\r\n`offline` mode, a `npm-shrinkwrap.json` file must be present during analysis.\r\n\r\n#### Added\r\n\r\n- *List added features here*\r\n\r\n#### Removed\r\n\r\n- *List removed features here*\r\n\r\n#### Fixed\r\n\r\n- *List fixed bugs here*\r\n\r\n#### Changed\r\n\r\n- Updated `.codeclimate.yml` configuration file to fix Code Climate builds\r\n- Generated and committed `npm-shinkwrap.jspn` file for Code Climate nodesecurity engine\r\n- Add `node_modules` folder to Code Climate exclude_paths\r\n\r\n#### Deprecated\r\n\r\n- *List any feature that is being deprecated in this*\r\n\r\n### Relevant Open Issues\r\n\r\n### Existing PRs with overlapping scope\r\n\r\n- #318 - feat(metrics): configuration for codeclimate\r\n\r\n### Screenshots of changes\r\n\r\n*(if modifying a view)*\r\n\r\n### Checklist\r\n\r\n- [x] This PR passes Linting tests (see below)\r\n- [x] This PR does not contain merge conflicts with `develop` (see below)\r\n- [x] This PR does not contain:\r\n  - Additional Dependencies in `package.json` that are not used\r\n  - Frontend JS in Jade templates (unless absolutely necessary)\r\n"},"comment":{"url":"https://api.github.com/repos/sfbrigade/brigadehub/issues/comments/223776974","html_url":"https://github.com/sfbrigade/brigadehub/pull/328#issuecomment-223776974","issue_url":"https://api.github.com/repos/sfbrigade/brigadehub/issues/328","id":223776974,"user":{"login":"therebelrobot","id":679178,"avatar_url":"https://avatars.githubusercontent.com/u/679178?v=3","gravatar_id":"","url":"https://api.github.com/users/therebelrobot","html_url":"https://github.com/therebelrobot","followers_url":"https://api.github.com/users/therebelrobot/followers","following_url":"https://api.github.com/users/therebelrobot/following{/other_user}","gists_url":"https://api.github.com/users/therebelrobot/gists{/gist_id}","starred_url":"https://api.github.com/users/therebelrobot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/therebelrobot/subscriptions","organizations_url":"https://api.github.com/users/therebelrobot/orgs","repos_url":"https://api.github.com/users/therebelrobot/repos","events_url":"https://api.github.com/users/therebelrobot/events{/privacy}","received_events_url":"https://api.github.com/users/therebelrobot/received_events","type":"User","site_admin":false},"created_at":"2016-06-04T20:37:01Z","updated_at":"2016-06-04T20:37:01Z","body":"@johngluck65, eslint, so long as it's following the same rules that feross/standard is based on, is perfectly fine for affecting GPA. That's part of what code health is, and since we're enforcing it, we should be able to show off the healthy status of our project :)"}}
{"actor":{"display_login":"therebelrobot"}}