-
Notifications
You must be signed in to change notification settings - Fork 88
Conversation
https://github.com/lazd/gulp-karma isn't a real plugin as it doesn't process any files. It has some fundamental design issues (lazd/gulp-karma#33) and doesn't bring enough value as Karma can be invoked directly (see https://github.com/karma-runner/gulp-karma)
@lazd is pretty good with gulp guidelines. We should ping him first |
Definitively! Although I had a number of exchanges with @lazd on the topic. I believe that with the current Karma design gulp-karma as a plugin is not providing any value and creates problem described in lazd/gulp-karma#21. This might change in the future when Karma exposes more hook points but this is not the case today. I hesitated a long time before opening this PR but today I had to debug another project where https://github.com/lazd/gulp-karma was used and "watch wasn't working" and this was one time too much. Sorry for putting it straight like this but we should not encourage people to use plugins that have well-known problems and a simpler alternative exists. |
@pkozlowski-opensource Thanks for taking the time to help out btw |
There isn't much we can do to Since |
So what is the conclusion on this? Blacklist or not? |
I guess @lazd argument is that there are shorcomings in Karma that - when solved - would make So, looking at where we stand today I still claim that |
@pkozlowski-opensource Yep that makes sense to me. Remove from the blacklist when it is ready. |
Agreed. @pkozlowski-opensource, let's see what we can do about making Karma play nice with gulp -- if there is an adaptor of sorts, it can live on as |
https://github.com/lazd/gulp-karma isn't a real plugin as it doesn't process any files.
It has some fundamental design issues (lazd/gulp-karma#33)
and doesn't bring enough value as Karma can be invoked directly
(see https://github.com/karma-runner/gulp-karma)