{"action":"created","issue":{"url":"https://api.github.com/repos/bbc/simorgh/issues/181","repository_url":"https://api.github.com/repos/bbc/simorgh","labels_url":"https://api.github.com/repos/bbc/simorgh/issues/181/labels{/name}","comments_url":"https://api.github.com/repos/bbc/simorgh/issues/181/comments","events_url":"https://api.github.com/repos/bbc/simorgh/issues/181/events","html_url":"https://github.com/bbc/simorgh/issues/181","id":335824070,"node_id":"MDU6SXNzdWUzMzU4MjQwNzA=","number":181,"title":"Add npm ci before test suite runs","user":{"login":"pjlee11","id":7791726,"node_id":"MDQ6VXNlcjc3OTE3MjY=","avatar_url":"https://avatars1.githubusercontent.com/u/7791726?v=4","gravatar_id":"","url":"https://api.github.com/users/pjlee11","html_url":"https://github.com/pjlee11","followers_url":"https://api.github.com/users/pjlee11/followers","following_url":"https://api.github.com/users/pjlee11/following{/other_user}","gists_url":"https://api.github.com/users/pjlee11/gists{/gist_id}","starred_url":"https://api.github.com/users/pjlee11/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pjlee11/subscriptions","organizations_url":"https://api.github.com/users/pjlee11/orgs","repos_url":"https://api.github.com/users/pjlee11/repos","events_url":"https://api.github.com/users/pjlee11/events{/privacy}","received_events_url":"https://api.github.com/users/pjlee11/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-06-26T13:38:06Z","updated_at":"2018-06-26T18:21:56Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"**Is your feature request related to a problem? Please describe.**\r\nWhen merging in latest to a feature branch any changes to the package.json do not affect my local env until I run an npm install. This can cause situations were tests/linting seem to be passing but the local setup is out of date.\r\n\r\n**Describe the solution you'd like**\r\nrun npm ci as a prerequisite to npm test.\r\n\r\n**Describe alternatives you've considered**\r\nnpm install instead of npm ci but this would also update the package-lock.json whereas npm ci will use the package-lock.json from latest.\r\n\r\n**Additional context**\r\nAdd any other context or screenshots about the feature request here.\r\n"},"comment":{"url":"https://api.github.com/repos/bbc/simorgh/issues/comments/400414496","html_url":"https://github.com/bbc/simorgh/issues/181#issuecomment-400414496","issue_url":"https://api.github.com/repos/bbc/simorgh/issues/181","id":400414496,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMDQxNDQ5Ng==","user":{"login":"benjaminhobbs","id":15235276,"node_id":"MDQ6VXNlcjE1MjM1Mjc2","avatar_url":"https://avatars0.githubusercontent.com/u/15235276?v=4","gravatar_id":"","url":"https://api.github.com/users/benjaminhobbs","html_url":"https://github.com/benjaminhobbs","followers_url":"https://api.github.com/users/benjaminhobbs/followers","following_url":"https://api.github.com/users/benjaminhobbs/following{/other_user}","gists_url":"https://api.github.com/users/benjaminhobbs/gists{/gist_id}","starred_url":"https://api.github.com/users/benjaminhobbs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benjaminhobbs/subscriptions","organizations_url":"https://api.github.com/users/benjaminhobbs/orgs","repos_url":"https://api.github.com/users/benjaminhobbs/repos","events_url":"https://api.github.com/users/benjaminhobbs/events{/privacy}","received_events_url":"https://api.github.com/users/benjaminhobbs/received_events","type":"User","site_admin":false},"created_at":"2018-06-26T18:21:56Z","updated_at":"2018-06-26T18:21:56Z","author_association":"CONTRIBUTOR","body":"I suppose the downside is that if package-lock.json on latest falls out of sync, it'll be harder to notice. I would suggest the npm install is used and then the user will see the lock change if there's any issue."}}
{"actor":{"display_login":"benjaminhobbs"}}