Skip to content

Add support for RDF4J 4#504

Merged
Ostrzyciel merged 5 commits intomainfrom
piotr/rdf4j-4
Aug 25, 2025
Merged

Add support for RDF4J 4#504
Ostrzyciel merged 5 commits intomainfrom
piotr/rdf4j-4

Conversation

@Ostrzyciel
Copy link
Member

This will allow us to use the Jelly plugin with applications still using RDF4J 4 – quite a few commercial DBs still do.

The changes are easy and should not be hard to maintain for a while.

I added a CI test to make sure it works with both 4 and 5.

@codecov
Copy link

codecov bot commented Aug 25, 2025

Codecov Report

❌ Patch coverage is 76.92308% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...elly/convert/rdf4j/rio/JellyIntegerRioSetting.java 40.00% 3 Missing ⚠️
Files with missing lines Coverage Δ Complexity Δ
...k/jelly/convert/rdf4j/rio/JellyParserSettings.java 60.00% <100.00%> (ø) 0.00 <0.00> (ø)
...k/jelly/convert/rdf4j/rio/JellyWriterSettings.java 53.12% <100.00%> (ø) 0.00 <0.00> (ø)
...elly/convert/rdf4j/rio/JellyIntegerRioSetting.java 40.00% <40.00%> (ø) 0.00 <0.00> (?)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Ostrzyciel Ostrzyciel enabled auto-merge (squash) August 25, 2025 19:05
@Ostrzyciel Ostrzyciel merged commit 7e7a133 into main Aug 25, 2025
11 checks passed
@Ostrzyciel Ostrzyciel deleted the piotr/rdf4j-4 branch August 25, 2025 19:07
Ostrzyciel added a commit that referenced this pull request Aug 25, 2025
Follow-up of #504

I forgot to update the version in build.sbt back to 5...
Ostrzyciel added a commit that referenced this pull request Aug 25, 2025
Follow-up of #504

I forgot to update the version in build.sbt back to 5...
Ostrzyciel added a commit that referenced this pull request Aug 25, 2025
Follow-up of #504

I forgot to update the version in build.sbt back to 5...
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