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.

Sign up for GitHub

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

Jump to bottom

Spec suggestions #4

Closed
wants to merge 1 commit into from
Closed

Spec suggestions #4

wants to merge 1 commit into from

Conversation

joepio
Copy link
Member

@joepio joepio commented Jun 4, 2020 •
edited
Loading

@joepio
Spec suggestions
b492eab 
- Support blank nodes in graph #2
- Default to URI datatype and drop the unresolving named node URL #1
@joepio joepio requested a review from rescribet June 4, 2020 11:48
@rescribet
Copy link
Contributor

These two are vastly different issues.

I really don't feel much for the implicit _: differentiation. Hinting towards being a valid scheme whilst being invalid. Any third-party platform URL parsers might accidentally allow them as valid URLs causing hard to track errors.

Leaving the DT empty as an implicit default also leaves room for errors, especially when other parsers default to string on the same code path.

Perhaps this strikes a balance?

@joepio joepio closed this Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rescribet rescribet Awaiting requested review from rescribet

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@joepio @rescribet