Skip to content

Make Jena/RDF4J deps provided, set default Jena to 6 - #747

Merged
Ostrzyciel merged 1 commit into
mainfrom
piotr/jena-6-inversion
Sep 28, 2026
Merged

Ostrzyciel merged 1 commit into
mainfrom
piotr/jena-6-inversion

Conversation

@Ostrzyciel

Copy link
Copy Markdown
Member
  • Set the default Jena version to 6 and change it to 5 in CI. This eliminates the constant false alarms about supposed CVEs that we don't have... It also makes it more reasonable to e.g., write benchmarks and tests because by default we benchmark and test latest Jena, not some buggy release from a year ago.
  • Make Jena and RDF4J deps "provided" so that users can safely downgrade Jena or RDF4J to some earlier version that they desire. This is what makes it possible to use Jena 5 with Jelly still. It also makes sense because usually Jelly is added as an extra dependency to Java projects, not as the "main thing that pulls in Jena".

Also set Jena's default version to 6 to avoid the constant false-alarm nagging about CVEs....
@codecov

codecov Bot commented Sep 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Ostrzyciel

Copy link
Copy Markdown
Member Author

I will fix the "required" checks after this PR is approved, that's a GH config issue

@Ostrzyciel
Ostrzyciel enabled auto-merge (squash) September 28, 2026 21:11
@Ostrzyciel
Ostrzyciel disabled auto-merge September 28, 2026 21:11
@Ostrzyciel
Ostrzyciel merged commit eaf73eb into main Sep 28, 2026
11 checks passed
@Ostrzyciel
Ostrzyciel deleted the piotr/jena-6-inversion branch September 28, 2026 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants