{"push_id":1306222850,"size":1,"distinct_size":1,"ref":"refs/heads/improve-checkin-process","head":"f9ff73a14b8858b91d7d1f4cc75a5402a53c195a","before":"4cf258225936924c7d9d339c311286e88f4c9ae2","commits":[{"sha":"f9ff73a14b8858b91d7d1f4cc75a5402a53c195a","author":{"name":"Phil Aquilina","email":"9e5380c23b28b93deaae0bcb9f4c7f8877236313@gmail.com"},"message":"Make checking in and publishing code simpler/more correct\n\nProblem:\nCurrently we check-in our transpiled code. Other than this being a\ngeneral nuisance, it's actually a little dangerous as well. Because it's\nessentially one big blob of code, it's hard to read and parse. We've\nalready encountered situations when someone's checked in code bashed\nsomeone else's changes.\n\nFix:\nIt's fairly simple to fix this by checking in only the source code and\nmaking sure we publish the build code.\n\nDetails:\nI've moved all our source code into /src, that way it's easier to target\nfor things like our linter. This change is actually straightforward\nsince all the paths are relative and therefore I only needed to update\nwebpack's config to point to /src/index.es6.js.\n\nI then updated package.json to have a prepublish hook to ensure that\nour source code is transpiled before it's pushed to npm.\n\nOther changes:\n\n- Removed a bunch of gitignore things that looked irrelevant to this\n  library\n- Removed .editorconfig and mancyLoader, which also looks irrelevant.\n- Updated package.json with more up to date information and updated\n  build to 0.10.0 (tested this out as well).\n- Added a bunch of stuff that shouldn't get published to npmignore.\n\nThere's some other funky stuff going on in this repo, such as js files\nbeing executable and the .es6.js extensions but in the interest of\nsmaller changes (and this is already a pretty big one), I'm not going to\ntouch those on this pass.","distinct":true,"url":"https://api.github.com/repos/phil303/node-api-client/commits/f9ff73a14b8858b91d7d1f4cc75a5402a53c195a"}]}
{"actor":{"display_login":"phil303"}}