-
-
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-72028] at test/src/test/java/jenkins/
replace deprecated URL()
#8570
Conversation
This reverts commit aeef4c1.
/label java |
/label hacktoberfest |
/label tests |
requesting @jtnord @timja @NotMyFault |
@jtnord I hope this is ready to merge PR |
failure ( |
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!
/label ready-for-merge This PR is now ready for merge. We will merge it after ~24 hours if there is no negative feedback. |
Replace deprecated
URL()
withURI
#.toURL()
. Changes implemented for test/src/test/java/jenkins/ files only.See JENKINS-72028.
Testing done
Proposed changelog entries
URL()
withURI#.toURL()
Proposed upgrade guidelines
N/A
Submitter checklist
Desired reviewers
@mention
Before the changes are marked as
ready-for-merge
:Maintainer checklist