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

WIP: added nested function (fixes #2) #3

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

Conversation

bjdmeest
Copy link
Contributor

@bjdmeest bjdmeest commented Apr 7, 2020

No description provided.

@pheyvaer
Copy link
Contributor

Does this still require changes?

@bjdmeest
Copy link
Contributor Author

I think that the example is correct, but also there's a bug in the parser RMLio/yarrrml-parser#68

@bjdmeest bjdmeest changed the title added nested function (fixes #2) WIP: added nested function (fixes #2) Apr 16, 2020
@pheyvaer
Copy link
Contributor

The bug is in the RMLMapper instead RMLio/rmlmapper-java#53

@pheyvaer
Copy link
Contributor

pheyvaer commented Oct 4, 2021

I guess we still can merge it based on the RMLMapper issue, right? @bjdmeest

@bjdmeest
Copy link
Contributor Author

bjdmeest commented Oct 4, 2021

well, I can just imagine people seeing and trying this example and then seeing it doesn't work, which might be confusing, I'd personally keep it open until the issue is fixed

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