-
Notifications
You must be signed in to change notification settings - Fork 117
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
Release grunt-karma 3.0 #261
Comments
@johnjbarton Thanks, will do. By the way, which integration tests are you referring to? |
@johnjbarton Tthe integration tests don't currently cover grunt-karma (I think). Is your intention to make just make sure we are testing and passing with the latest version of Karma (devDeps), or were you referring the peerDeps support requirement? One the projects I work on uses grunt-karma 2.0 with karma 2.x, and while it runs fine, it surprises me that it installs without warnings. I'd expect npm to warn about the specified version in package.json (karma@2.0.2) not satisfying grunt-karma's Can we / should we continue to support the older versions, or add 3.x to the pipe? |
I think new the top priority is for new versions of grunt-karma to support the newest versions of karma-runner. For most cases that is enough. There may be circumstances where mixing versions is needed, but I don't think the extra work is justified compared to other priorities. |
The next release of grunt-karma will not support being used as wrapper to run karma 1 or older. Ref #261.
Sorry I can't get to this until ~24th. I tried today but ran into some issues with login and I will be out on vacation. |
3.0 is up |
@Krinkle assign this to me when you are ready (the release scripts have some issues I am trying to iron).
IMO the package.json should dep karma ^3.0.0 since that is where the integration tests run.
The text was updated successfully, but these errors were encountered: