Skip to content
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

Pushed Jena to v5.2.0 #184

Merged
merged 15 commits into from
Jan 10, 2025
Merged

Pushed Jena to v5.2.0 #184

merged 15 commits into from
Jan 10, 2025

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
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