Skip to content
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

Removed peer dependencies. #846

Closed

Conversation

andersjanmyr
Copy link

Peer dependencies should not go from the parent to the children. The children
should peer-depend on the parent project and not the other way around.

@vojtajina
Copy link
Contributor

Yep, there's an issue for this #767. We need to update karma init to install plugins, then I'm fine with removing this.

So, can you please:

  • correct the commit msg to follow karma-runner.github.io/0.10/dev/git-commit-msg.html
  • add Closes #767 into the commit msg body
  • add a BREAKING CHANGE: description, explaining that people have to explicitly install all the plugins they need and that karma init is suppose to help with it.

Peer dependencies should not go from the parent to the children. The children
should peer depend on the parent project and not the other way around.
Closes karma-runner#767

BREAKING CHANGE:
People will need to explicitly install the plugins they need. `karma init`
should help with this.

Removed plugins that need to be installed explicitly are:

* karma-jasmine
* karma-requirejs
* karma-coffee-preprocessor
* karma-html2js-preprocessor
* karma-chrome-launcher
* karma-firefox-launcher
* karma-phantomjs-launcher
* karma-script-launcher
@andersjanmyr
Copy link
Author

Updated the commit message as @vojtajina suggested.

@vojtajina
Copy link
Contributor

I updated the msg and merged it as e033d56.

Thanks @andersjanmyr

@vojtajina vojtajina closed this Dec 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants