-
Notifications
You must be signed in to change notification settings - Fork 133
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
Make dep on matrix-project
optional
#287
Conversation
Thanks. I've open some issues on other popular plugin to not force install of the matrix-project which I don't use on our pipeline only jenkins https://issues.jenkins.io/browse/JENKINS-73457 Not sure how difficult i would be to remove the hard dependency |
Causes PCT failures that are reproducible with
|
So that a Jenkins installation which is 100% Pipeline can not bother having this plugin installed. Amends 6204dbf. The purported fix seems very suspect to me (
Item.onLoad
should be called only by Jenkins core, not in this context) and there is no regression test so I am half inclined just to delete it.