-
Notifications
You must be signed in to change notification settings - Fork 455
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
RDF Formatter Contribution #2261
RDF Formatter Contribution #2261
Conversation
FYI: Expecting a dependency update for turtle-formatter soon-ish (PR already submitted). If it is allowed by your review process, I'd bump that dependency as it becomes available. EDIT: that's done. |
Looks great! Needs an entry in the |
Comparing graph nodes using their `equals()` method is used under the hood by the isomorphicity check and therefore has to be used in the diff calcluation as well (otherwise the latter might not find any differences). Special handling of blank nodes has been omitted as it is hard to find which ones are different and which ones are not, so we just print them all for now.
I think it's done now. |
@nedtwigg ... what do I do? rebase and force-push ? |
Nope, I just fixed the merge conflict, if CI passes this should be good to go. |
Run |
Done. |
Limited the tests to JDK >= 17 to deal with the tests failing for java 11. Hope that's it now. |
@nedtwigg could you please approve the workflow runs? Thanks! |
Released in |
Adds a formatter for RDF
In its current form, only the TTL format can be formatted using turtle-formatter. All other formats could be handled with Jena, but are currently not implemented.
The PR contains code for a maven plugin, but not a gradle/sbt plugin.
Options are:
The PR includes smoke tests that use input/output folders containing files with identical names, for which a utility method was added to the ResourceHarness.