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

add SHACL definition validation in CI #1057

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

jimkont
Copy link
Contributor

@jimkont jimkont commented Dec 6, 2024

There were some invalid SHACL declarations in the constraint definition. With this PR, I removed them and, additionally, added validation of all the SHACL files using shacl-shacl. The format plugin did not let me keep some comments on the SHACL-SHACL.ttl file wrt origin, not sure if you want to add some details in a readme file or if it is ok as is

@fkleedorfer
Copy link
Collaborator

Yes, the format plugin removes any comments. Future work.

@fkleedorfer
Copy link
Collaborator

Love this, btw!

@fkleedorfer fkleedorfer merged commit ecb3503 into qudt:main Dec 6, 2024
1 check passed
@jimkont jimkont deleted the fix/shacl-conformance branch December 6, 2024 09:06
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