-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
It should be much faster.. #78
Comments
I'm not sure I follow... Can you try to explain in more detail? |
If you talking about |
No I'm already using jit-grunt. Without its even slower. With load-grunt-config the loading-tasks task takes about 1s (using time-grunt). With the method described above it takes about 200-400 miliseconds. |
@jgallen23 correct me if I'm wrong, but as I know @johannesjo sorry, but I don't get what is your main point in this issue. :( |
@SolomoN-ua The whole point is to show, that the module is way slower than it should be. The modules readconfigdir.js should be rewritten to be as fast, as it could be (only in my humble opinion of course). Also jit-grunt is much faster. |
Hi @johannesjo, can you provide a PR for this? |
With this little function the watch task is about 3 to 4 times faster, as it is with load-grunt-config:
The text was updated successfully, but these errors were encountered: