{"action":"created","issue":{"url":"https://api.github.com/repos/browserify-rails/browserify-rails/issues/43","labels_url":"https://api.github.com/repos/browserify-rails/browserify-rails/issues/43/labels{/name}","comments_url":"https://api.github.com/repos/browserify-rails/browserify-rails/issues/43/comments","events_url":"https://api.github.com/repos/browserify-rails/browserify-rails/issues/43/events","html_url":"https://github.com/browserify-rails/browserify-rails/issues/43","id":58251869,"number":43,"title":"Performance","user":{"login":"cymen","id":121034,"avatar_url":"https://avatars.githubusercontent.com/u/121034?v=3","gravatar_id":"","url":"https://api.github.com/users/cymen","html_url":"https://github.com/cymen","followers_url":"https://api.github.com/users/cymen/followers","following_url":"https://api.github.com/users/cymen/following{/other_user}","gists_url":"https://api.github.com/users/cymen/gists{/gist_id}","starred_url":"https://api.github.com/users/cymen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cymen/subscriptions","organizations_url":"https://api.github.com/users/cymen/orgs","repos_url":"https://api.github.com/users/cymen/repos","events_url":"https://api.github.com/users/cymen/events{/privacy}","received_events_url":"https://api.github.com/users/cymen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-02-19T18:56:14Z","updated_at":"2015-06-20T16:12:40Z","closed_at":null,"body":"So we have performance issues sometimes. I want things to work faster locally and I am seeing some potential areas to make things faster:\r\n\r\n* when source maps are enabled, everything gets source maps including node_modules -- I could live without source maps for node_modules by default with option to turn them on\r\n* browserify supports a `noparse` option -- configuring it manually is annoying but automating it seems risky too -- potentially, we could do a noparse if we had a cache that told us if a file was CommonJS or not (if it needed to be browserified)\r\n* browserify has a `--fast` option -- it would be good to know if \r\n\r\nFor caching, we need something in memory (or in filesystem, I'd prefer memory). That cache would need the last modified time for a file along with other things. This would only be present for development -- in production with Rails, the assets are precompiled so browserify-rails should never need to be running in production (which frees us up on how we implement this quite a bit, let me know if my assumptions are wrong here).\r\n\r\nThis isn't comprehensive but I want to get the conversation going and see if anyone else is thinking about this (I know @gabrielecirulli is -- thanks to him, browserify-incremental is in with https://github.com/browserify-rails/browserify-rails/pull/35)."},"comment":{"url":"https://api.github.com/repos/browserify-rails/browserify-rails/issues/comments/113787096","html_url":"https://github.com/browserify-rails/browserify-rails/issues/43#issuecomment-113787096","issue_url":"https://api.github.com/repos/browserify-rails/browserify-rails/issues/43","id":113787096,"user":{"login":"ciaoben","id":4526061,"avatar_url":"https://avatars.githubusercontent.com/u/4526061?v=3","gravatar_id":"","url":"https://api.github.com/users/ciaoben","html_url":"https://github.com/ciaoben","followers_url":"https://api.github.com/users/ciaoben/followers","following_url":"https://api.github.com/users/ciaoben/following{/other_user}","gists_url":"https://api.github.com/users/ciaoben/gists{/gist_id}","starred_url":"https://api.github.com/users/ciaoben/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ciaoben/subscriptions","organizations_url":"https://api.github.com/users/ciaoben/orgs","repos_url":"https://api.github.com/users/ciaoben/repos","events_url":"https://api.github.com/users/ciaoben/events{/privacy}","received_events_url":"https://api.github.com/users/ciaoben/received_events","type":"User","site_admin":false},"created_at":"2015-06-20T16:12:40Z","updated_at":"2015-06-20T16:12:40Z","body":"I was trying the gem, and I was wondering if I am missing, or doing something wrong, but it seems non-usable to me:\r\n\r\nI've only 2 deps for now: ´martyjs´ and ´react´. \r\n\r\napplication.js:\r\n´´´\r\n...\r\n//= require turbolinks\r\nvar Marty = require(\"marty\");\r\nconsole.log(Marty);\r\n´´´\r\n\r\nI refresh the browser, and it makes 30 secs to bundle up everything, and its good for the first time.\r\n\r\nThen I refresh after simply having added a ´console.log(\"hi\");´ and it takes again the same time that it takes the first time.\r\n\r\nIf I have understood correctly, with browserify incremental this time should have evaluated only the new line. But performace doesn't say so.\r\n\r\n\r\nAm I wrong?"}}