-
Notifications
You must be signed in to change notification settings - Fork 177
fix URL constructor deprecation warning #627
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
Conversation
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. |
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.
This line is too long, can we format this line only.
f82aaa9
to
df60825
Compare
fixes the constructor deprecation of `java.net.URL` that first appeared in java 20 by replacing it with `java.net.URI.toURL()`
2e87bdf
to
70738af
Compare
stub/build.gradle.kts
Outdated
} | ||
|
||
tasks.withType<DokkaTask>().configureEach { | ||
|
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.
Remove this empty line.
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.
should be fixed now!
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). |
fixes the constructor deprecation warning of
java.net.URL
that first appeared in java 20 by replacing it withjava.net.URI.toURL()
closes #626