Skip to content

Conversation

wavedeck
Copy link
Contributor

fixes the constructor deprecation warning of java.net.URL that first appeared in java 20 by replacing it with java.net.URI.toURL()

closes #626

Copy link

linux-foundation-easycla bot commented Sep 14, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@duckladydinh
Copy link
Collaborator

This PR somehow doesn't trigger tests. Can you make rebase and make some empty commit to see if we can trigger them? Without successful required tests, it can't be merged.

@duckladydinh duckladydinh self-assigned this Aug 24, 2025
Copy link
Collaborator

@duckladydinh duckladydinh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is too long, can we format this line only.

@wavedeck wavedeck force-pushed the issue-626-deprecated-url branch from f82aaa9 to df60825 Compare August 25, 2025 13:52
fixes the constructor deprecation of `java.net.URL` that first appeared in java 20 by replacing it with `java.net.URI.toURL()`
@wavedeck wavedeck force-pushed the issue-626-deprecated-url branch 2 times, most recently from 2e87bdf to 70738af Compare August 25, 2025 14:12
@wavedeck wavedeck requested a review from duckladydinh August 25, 2025 14:15
}

tasks.withType<DokkaTask>().configureEach {

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this empty line.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be fixed now!

@duckladydinh duckladydinh merged commit d48bc3e into grpc:master Aug 25, 2025
4 checks passed
@duckladydinh
Copy link
Collaborator

Thanks, referecing #618 (comment), I am active for a few weeks here so if you still have something you want to fix, now is the best time of the year ;) (only short changes though).

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.

java 20 / 21 - constructor URL(String!) is deprecated

2 participants