-
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
peerDeps still point at older karma #267
Comments
@gkatsev Thanks, yeah, it was planned and is pending review at #264. In my own testing, it seems to actually work fine because npm no longer downloads peerDependencies (which is good), instead it is the project's own responsibility to install the preferred version. As such, if you specify karma 3.0.0 with grunt-karma 3.0.0, then it will actually use those together, it shouldn't install karma 1.x, and it also shouldn't refuse to use the karma 3.0 version. But, of course, it does cause a warning, and that's wrong. We instead should print a warning when the project is using a karma version that is not 3.x. One the linked PR lands, I recommend we do a minor release to make it available. |
Ah, must've missed that PR. Yeah, npm no longer forces peerDeps which is good. Thanks for the info! |
It works fine, but when running
Great if #264 could be released! |
Done! |
Will there be a patch release soon to reflect these changes? |
It was great to see a new release of grunt-karma, looking into it, it seems that the peerDependencies are still pointing at karma 0.13 and 1.0 https://github.com/karma-runner/grunt-karma/blob/v3.0.0/package.json#L50-L53 and https://registry.npmjs.org/grunt-karma/3.0.0
It sounds like that was the plan for this release (#261), though, maybe there's some background I'm missing.
Thanks!
The text was updated successfully, but these errors were encountered: