-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Build of the app is too slow #1094
Comments
Gulp, I guess. I wonder how many we'll improve with the planned switch to gulp. (#837). Though other improvements are also welcome. #1089 might also make sense to you.
All contributions are welcome! Check the open issues, but mainly #553. |
its not a switch but an option to choose either.
This is computing time here. To make minification faster would be up to uglify to optimize. There isn't much we can to improve that. Look at how long compiling C or Java things take. Whats fast to you might be slow to others so this is very subjective. Waiting a few seconds to me is acceptable when the end result is so nice |
@eddiemonge how are you sure that is computing time that causes my build lasting 5minutes? From the comment you cited, it looks like it is the opposite (because of grunt). Indee the same number of files is much faster using only raw uglify2. I would be really eager to help you switch to gulp or add integration (once I can prove that the slowness is because of it). |
Can you put together a test using grunt with only uglify and see you can reproduce the 'slowness'. This is probably better answered on grunt-contrib-uglify though. Switching to gulp won't really help in this situation. Either its a bug in gcu or uglify is slow and it won't matter because gulp will have the same issues. |
It is mainly an issue from uglify2, i described it here (tl;dr: uglify2 is too slow when files are too many. these files are not already minified).
But because you take care of how the build process runs (and you choose the module inside Gruntfile) I was thinking it is a good thing to notify this issue also here.
I was wondering:
The text was updated successfully, but these errors were encountered: