gulp-karma icon indicating copy to clipboard operation
gulp-karma copied to clipboard

Upgrading to support newer karma versions.

Open gautema opened this issue 9 years ago • 3 comments

I have upgraded all dependencies and fixed so it don't get deprecated messages from karma. And will work even in next karma version.

gautema avatar Jul 30 '15 07:07 gautema

@lazd Any way we can get this merged and pushed to npm?

kevcenteno avatar Aug 18 '15 19:08 kevcenteno

@kevcenteno honestly, this plugin needs to be deprecated. See #43.

The way things are done here is error prone, not gulpish, and just quite a bit hacky. When I built this, gulp was young and I needed a way to use it with Karma. Since then, the Karma team has fixed many problems I was working around here (the whole notion of background.js) and come up with a recommended way to use Karma with gulp.

Can you please take a look at the recommended approach and let me know what shortcomings are causing you to still use gulp-karma? We can work with the Karma team to find a way around them.

lazd avatar Aug 18 '15 19:08 lazd

Thank you for the quick response, @lazd.

The main thing I like this gulp task is that I can use wiredep to load my dependencies (whether they're from bower or somewhere else). Adding a bower wildcard path in the karma conf gets weird real fast (probably due to ordering), so we end up having to list out the deps explicitly.

kevcenteno avatar Aug 18 '15 19:08 kevcenteno