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

fix cardinalities #638

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix cardinalities #638

wants to merge 2 commits into from

Conversation

jeswr
Copy link

@jeswr jeswr commented Mar 26, 2024

Resolves #637

@labra
Copy link
Member

labra commented Mar 27, 2024

Thanks for the pull request. I noticed that 3 tests are failing, and I think they can be solved changing the expected results. Do you want to try to change those tests also? If not, I can also try it, but it would take me some time (at least 2 weeks...).

@jeswr
Copy link
Author

jeswr commented Mar 27, 2024

I'm not in a rush to have this fixed as it is monkey patched in my use case, and I also know very little Scala and don't have a good dev environment for Scala set up sp it may be better for you to fix it when time allows.

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.

shacl -> shex cardinality bugs
2 participants