-
Notifications
You must be signed in to change notification settings - Fork 12
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
Make termType values a closed set #121
Comments
I propose changing the following sentence describing a Term:
to
or
|
To make it an absolute requirement we should define it using MUST https://tools.ietf.org/html/rfc2119 |
@bergos you wrote all the WebIDL in ReSpec version, I wonder if we should use WebIDL Enumerations enum TermType { "NamedNode", "BlankNode", "Literal", "Variable", "DefaultGraph }; interface Term {
attribute TermType termType;
attribute string value;
boolean equals(Term other);
}; |
Well, what happens when you extend a term to be able to be a nested graph? Maybe add "Graph" to the list now? |
@timbl does it mean that in a
How does a "nested graph" differ from having a graph named with IRI (NamedNode) or BlankNode and Quads just using those names to reference them instead of "nesting"? |
+1 to make to having a closed set. |
Following https://lists.w3.org/Archives/Public/public-rdfjs/2017Sep/0003.html, I suggest we make the language more strict and only allow those 5 values.
The text was updated successfully, but these errors were encountered: