-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add uses directive #11761
Add uses directive #11761
Conversation
This is for merged module, neecs to be adjusted in the gradle file |
In addition I guess... |
The build for this PR is no longer available. Please visit https://builds.jabref.org/main/ for the latest build. |
@InAnYan Check the diff. Interesting learning about Java modules... |
The issues arose because the djl library is not modularized properly, so it needs to be put in the merged module ("merged module") and as it uses service loaders, we need to specify the implementations as well |
Is there already an issue at that repo @InAnYan ? |
Oh, I see. Great that you worked it out! |
Reported by @Siedlerchr
Mandatory checks
CHANGELOG.md
described in a way that is understandable for the average user (if applicable)