Skip to content

Pushed Jena to v5.2.0#184

Merged
ashleycaselli merged 15 commits intomasterfrom
177-jena5
Jan 10, 2025
Merged

Pushed Jena to v5.2.0#184
ashleycaselli merged 15 commits intomasterfrom
177-jena5

Conversation

@ashleycaselli
Copy link
Collaborator

This PR contains the Jena migration from v4 to v5 (done by @jerdebsp, thanks @afs for the hints and code review).
I then pushed Jena to v5.2.0 and fixed all the code-deprecated warnings.

Also, the docker image is now built with Java v21 as the base image. Maven tests are run on both Java v17 and v21. The code is still compiled by the maven-compiler-plugin against Java v17.

@ashleycaselli ashleycaselli added the dependencies Pull requests that update a dependency file label Dec 18, 2024
@jerdebsp
Copy link
Contributor

jerdebsp commented Jan 7, 2025

Couldn't figure out the changes in [655cd7e] but the rest seems fine. I tried Jena 5.2.0 on my private branch and works good

@ashleycaselli
Copy link
Collaborator Author

Couldn't figure out the changes in [655cd7e] but the rest seems fine. I tried Jena 5.2.0 on my private branch and works good

That commit is mostly for code-style cleaning/refactoring. There were many code style warnings, i.e. using old Java versions syntax, using tabs instead of spaces, unused imports, mixed LF/CRLF, deprecated functions used.

@ashleycaselli ashleycaselli merged commit 7c73739 into master Jan 10, 2025
4 checks passed
@ashleycaselli ashleycaselli deleted the 177-jena5 branch July 10, 2025 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants