Migrate to new URI API#11824
Conversation
There was a problem hiding this comment.
Your code currently does not meet JabRef's code guidelines.
We use Checkstyle to identify issues.
The tool reviewdog already placed comments on GitHub to indicate the places. See the tab "Files" in you PR.
Please carefully follow the setup guide for the codestyle.
Afterwards, please run checkstyle locally and fix the issues.
You can check review dog's comments at the tab "Files changed" of your pull request.
|
seems like it just didn't clean up the imports |
|
still some errros (java:2496): dbind-WARNING **: 10:50:50.477: AT-SPI: Error retrieving accessibility bus address: org.freedesktop.DBus.Error.ServiceUnknown: The name org.a11y.Bus was not provided by any .service files
Test isURLshouldRejectInvalidURL() FAILED java.lang.IllegalArgumentException: URI is not absolute Test isURLshouldRejectEmbeddedURL() FAILED java.lang.IllegalArgumentException: Illegal character in scheme name at index 4: dblp computer science bibliography, http://dblp.org/ |
|
The build for this PR is no longer available. Please visit https://builds.jabref.org/main/ for the latest build. |
Applies https://docs.openrewrite.org/recipes/java/migrate/net/urlconstructorstourirecipes
This reduces compilation warnings.
Mandatory checks
CHANGELOG.mddescribed in a way that is understandable for the average user (if applicable)