{"action":"created","issue":{"url":"https://api.github.com/repos/este/este/issues/573","labels_url":"https://api.github.com/repos/este/este/issues/573/labels{/name}","comments_url":"https://api.github.com/repos/este/este/issues/573/comments","events_url":"https://api.github.com/repos/este/este/issues/573/events","html_url":"https://github.com/este/este/pull/573","id":115707400,"number":573,"title":"Remove gulp dependecy","user":{"login":"elmariofredo","id":82104,"avatar_url":"https://avatars.githubusercontent.com/u/82104?v=3","gravatar_id":"","url":"https://api.github.com/users/elmariofredo","html_url":"https://github.com/elmariofredo","followers_url":"https://api.github.com/users/elmariofredo/followers","following_url":"https://api.github.com/users/elmariofredo/following{/other_user}","gists_url":"https://api.github.com/users/elmariofredo/gists{/gist_id}","starred_url":"https://api.github.com/users/elmariofredo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/elmariofredo/subscriptions","organizations_url":"https://api.github.com/users/elmariofredo/orgs","repos_url":"https://api.github.com/users/elmariofredo/repos","events_url":"https://api.github.com/users/elmariofredo/events{/privacy}","received_events_url":"https://api.github.com/users/elmariofredo/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2015-11-08T03:35:43Z","updated_at":"2016-01-10T19:37:58Z","closed_at":"2015-12-19T01:24:47Z","pull_request":{"url":"https://api.github.com/repos/este/este/pulls/573","html_url":"https://github.com/este/este/pull/573","diff_url":"https://github.com/este/este/pull/573.diff","patch_url":"https://github.com/este/este/pull/573.patch"},"body":"WARNING: This is not meant to be merged, but rather to open discussion about necessity of gulp dependency in este and webpack projects in general. In este project is gulp used in a way that it's duplicating npm scripts functionality. Beside duplicity it's hard to find out what is actually happening behind each task. So I have tried to remove gulp dependency to solve these issues.\r\n\r\nI had to do these changes to make it work without gulp\r\n\r\n1. drop es6 imports in some of the scripts as babel-node is terribly slow\r\n2. remove from `src/server/index.js` `NODE_ENV` check as for development it is not set\r\n3. add `rctapp` dependecy which contain `clear-react-packager-cache` task\r\n4. replace `mochaRunCreator.js` with `mocha`\r\n\r\nI believe that these changes make project bit more readable and also reducing amount of code written for este which is one of the goals.\r\n\r\nLet me know what do you think."},"comment":{"url":"https://api.github.com/repos/este/este/issues/comments/170384241","html_url":"https://github.com/este/este/pull/573#issuecomment-170384241","issue_url":"https://api.github.com/repos/este/este/issues/573","id":170384241,"user":{"login":"steida","id":66249,"avatar_url":"https://avatars.githubusercontent.com/u/66249?v=3","gravatar_id":"","url":"https://api.github.com/users/steida","html_url":"https://github.com/steida","followers_url":"https://api.github.com/users/steida/followers","following_url":"https://api.github.com/users/steida/following{/other_user}","gists_url":"https://api.github.com/users/steida/gists{/gist_id}","starred_url":"https://api.github.com/users/steida/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/steida/subscriptions","organizations_url":"https://api.github.com/users/steida/orgs","repos_url":"https://api.github.com/users/steida/repos","events_url":"https://api.github.com/users/steida/events{/privacy}","received_events_url":"https://api.github.com/users/steida/received_events","type":"User","site_admin":false},"created_at":"2016-01-10T19:37:57Z","updated_at":"2016-01-10T19:37:57Z","body":"@elmariofredo The thing is, we need Gulp at least for little helpers like https://github.com/este/este/blob/master/gulpfile.babel.js#L90 Gulp is cross-platform, without Gulp, we would use various helpers like parallelshell anyway. Let's wait for webpack 2, maybe we can rethink Este approach.\r\n\r\n"}}