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

made activitystreams2.owl pass turtle-validator #66

Merged
merged 2 commits into from Jan 22, 2015

Conversation

elf-pavlik
Copy link
Member

noticed in #65
validated with https://github.com/mmlab/TurtleValidator

@elf-pavlik
Copy link
Member Author

@jasnell do you see any problem with merging this one? i share links to this file which according to validator currently has invalid turtle 😞

you may also like to check out this small nuance in turtle rdfjs/N3.js#30 (comment)

@elf-pavlik
Copy link
Member Author

@RubenVerborgh would you say with confidence that we can rely on this turtle-validator?
I mostly would like to double check that blank nodes shouldn't use . before closing square bracket. Most errors came from that so also most of my edits just remove this .

@RubenVerborgh
Copy link
Member

I actually didn't know about this validator, even though it's written by the guy next to me 😄

That said, yes you can rely on this, as N3.js passes all Turtle spec-compliance tests and is battle-tested.
I upgraded to the latest N3.js version, which includes several bugfixes.

@RubenVerborgh
Copy link
Member

And here's a patch to N3.js in case your blank nodes are really broken (using a . inside of them and omitting a closing bracket ]).

jasnell added a commit that referenced this pull request Jan 22, 2015
made activitystreams2.owl pass turtle-validator
@jasnell jasnell merged commit 10fb373 into w3c:master Jan 22, 2015
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.

None yet

3 participants