-
-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
[JENKINS-71666] Adapt to Popper deprecation in Jenkins core #8810
Conversation
Yay, your first pull request towards Jenkins core was created successfully! Thank you so much! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
This PR is now ready for merge. We will merge it after approximately 24 hours if there is no negative feedback. |
Congratulations on getting your very first Jenkins core pull request merged 🎉🥳 |
In general, any changes to the list of detached plugins should be tested as in e.g. #8724:
As a one-time courtesy, I have run this test locally and verified that it passed. In the future, please ensure that proper testing is done as part of the review process. |
Will do. Thanks! |
See JENKINS-71666. It removes the deprecated Popper api from war/pom.xml.
Testing done
Note: Only light-test was performed
Proposed changelog entries
Proposed upgrade guidelines
N/A
Submitter checklist
Desired reviewers
@MarkEWaite @NotMyFault
Before the changes are marked as
ready-for-merge
:Maintainer checklist