-
Notifications
You must be signed in to change notification settings - Fork 17
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
Migrate to Jena 4.10 #389
Migrate to Jena 4.10 #389
Conversation
632fbf4
to
b8eb3a4
Compare
Signed-off-by: Andrew Berezovskyi <[email protected]>
f578c6d
to
1b3ff4a
Compare
Signed-off-by: Andrew Berezovskyi <[email protected]>
Signed-off-by: Andrew Berezovskyi <[email protected]>
Just migrated from Jena 4.10 SNAPSHOT to 4.11 SNAPSHOT, build succeeds on 17 and 21. Jena now publishes 5.0 SNAPSHOT too, we could test how we fare. |
Signed-off-by: Andrew Berezovskyi <[email protected]>
Bad news, |
Signed-off-by: Andrew Berezovskyi <[email protected]>
Before I forget: seems to pass tests on refimpl and client samples: https://github.com/oslc-op/refimpl/actions/runs/11447506237 and https://github.com/OSLC/lyo-samples/actions/runs/11447508224 |
Description
Switch to the latest Jena 4.x (4.10.0) to anticipate Jena 5 breaking changes. So far we only had to replace usage of the shaded Guava classes (my fault for using them in the first place) and use
ReifierStd
instead ofStatement.listReifiedStatements()
.This PR also deals with JDK 17 warnings where possible.
Checklist
Issues
Closes #0; Closes #00
(use exactly this syntax to link to issues, one issue per statement only!)