Skip to content

fix: remove rdf4j-generator #715

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

Conversation

PatrickDekkerHealthRI
Copy link
Contributor

@PatrickDekkerHealthRI PatrickDekkerHealthRI commented Jun 5, 2025

Remove the dependency on (archived) rdf4j-generator, by removing the plugin from the maven pom.file.
And moved the generated source files to the normal source files.

this fixes #612 and fixes #714 (workaround)

@dennisvang dennisvang self-requested a review June 5, 2025 14:02
@dennisvang
Copy link
Contributor

Thanks @PatrickDekkerHealthRI that looks good.

I wonder if we can remove the unused classes and terms.
For example, SCHEMAORG appears to be completely unused, and from the others only a few terms are in use.

Copy link
Contributor

@dennisvang dennisvang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great. Thanks! :)

@dennisvang dennisvang merged commit 0230bae into FAIRDataTeam:support/1.17.x Jun 5, 2025
11 of 13 checks passed
@dennisvang dennisvang mentioned this pull request Jun 5, 2025
dennisvang pushed a commit that referenced this pull request Jun 5, 2025
* fix: remove rdf4j-generator

* fix: checkstyle warnings on DATACITE,FDP, R3D generate classes and remove SCHEMAORG
dennisvang added a commit that referenced this pull request Jun 10, 2025
* cherry-pick fix: remove rdf4j-generator (#715)

* fix: remove rdf4j-generator

* fix: checkstyle warnings on DATACITE,FDP, R3D generate classes and remove SCHEMAORG

* cherry-pick: Clean up vocabulary classes (#717)

* remove unused WebAccessControl vocabulary

* rename vocabulary.Sio to SIO and use util.ValueFactoryHelper.i for consistency

* remove unused terms from DATACITE vocabulary and refactor for consistency

* remove unused terms from R3D vocabulary and refactor for consistency

* remove unused terms from FDP vocabulary and refactor for consistency

---------

Co-authored-by: Patrick Dekker <168626347+PatrickDekkerHealthRI@users.noreply.github.com>
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