{"action":"created","issue":{"url":"https://api.github.com/repos/MiguelCastillo/bit-loader/issues/167","labels_url":"https://api.github.com/repos/MiguelCastillo/bit-loader/issues/167/labels{/name}","comments_url":"https://api.github.com/repos/MiguelCastillo/bit-loader/issues/167/comments","events_url":"https://api.github.com/repos/MiguelCastillo/bit-loader/issues/167/events","html_url":"https://github.com/MiguelCastillo/bit-loader/pull/167","id":105132755,"number":167,"title":"Add local grunt-cli, build before tests","user":{"login":"IanVS","id":4616705,"avatar_url":"https://avatars.githubusercontent.com/u/4616705?v=3","gravatar_id":"","url":"https://api.github.com/users/IanVS","html_url":"https://github.com/IanVS","followers_url":"https://api.github.com/users/IanVS/followers","following_url":"https://api.github.com/users/IanVS/following{/other_user}","gists_url":"https://api.github.com/users/IanVS/gists{/gist_id}","starred_url":"https://api.github.com/users/IanVS/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/IanVS/subscriptions","organizations_url":"https://api.github.com/users/IanVS/orgs","repos_url":"https://api.github.com/users/IanVS/repos","events_url":"https://api.github.com/users/IanVS/events{/privacy}","received_events_url":"https://api.github.com/users/IanVS/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-09-07T02:12:14Z","updated_at":"2015-09-07T02:35:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/MiguelCastillo/bit-loader/pulls/167","html_url":"https://github.com/MiguelCastillo/bit-loader/pull/167","diff_url":"https://github.com/MiguelCastillo/bit-loader/pull/167.diff","patch_url":"https://github.com/MiguelCastillo/bit-loader/pull/167.patch"},"body":"It's polite to those of us who do not have `grunt-cli` installed globally, to install locally in devDependencies.  :)\r\n\r\nThis also now performs a build before running tests, since without it the tests were failing on my box. Plus now you'll get linting prior to the tests.  "},"comment":{"url":"https://api.github.com/repos/MiguelCastillo/bit-loader/issues/comments/138162314","html_url":"https://github.com/MiguelCastillo/bit-loader/pull/167#issuecomment-138162314","issue_url":"https://api.github.com/repos/MiguelCastillo/bit-loader/issues/167","id":138162314,"user":{"login":"MiguelCastillo","id":1457701,"avatar_url":"https://avatars.githubusercontent.com/u/1457701?v=3","gravatar_id":"","url":"https://api.github.com/users/MiguelCastillo","html_url":"https://github.com/MiguelCastillo","followers_url":"https://api.github.com/users/MiguelCastillo/followers","following_url":"https://api.github.com/users/MiguelCastillo/following{/other_user}","gists_url":"https://api.github.com/users/MiguelCastillo/gists{/gist_id}","starred_url":"https://api.github.com/users/MiguelCastillo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MiguelCastillo/subscriptions","organizations_url":"https://api.github.com/users/MiguelCastillo/orgs","repos_url":"https://api.github.com/users/MiguelCastillo/repos","events_url":"https://api.github.com/users/MiguelCastillo/events{/privacy}","received_events_url":"https://api.github.com/users/MiguelCastillo/received_events","type":"User","site_admin":false},"created_at":"2015-09-07T02:35:17Z","updated_at":"2015-09-07T02:35:17Z","body":"Interesting.  I don't believe this is quite correct.\r\n\r\nYou need grunt-cli globally so that `grunt` is available as a command in your system.  Are you able to run `grunt` without installing `grunt-cli` globally?  Installing `grunt-cli` locally should throw a warning...\r\nhttp://gruntjs.com/getting-started#installing-the-cli\r\n\r\nAs far as the changes to package.json scripts...  Transpilation/bundling should be done at the `prepublish` step.  https://docs.npmjs.com/misc/scripts#common-uses. Overriding the `build` is really not very common for JS projects.\r\n\r\n** Really interested to hear some background on your changes."}}