-
Notifications
You must be signed in to change notification settings - Fork 2
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
Standardise a baseIRI parameter on import #18
Comments
Just to note, several parsers (N3, RDF/XML, JSON-LD, RDFa) have this parameter on the constructor, instead of the |
That is already there: https://github.com/rdfjs/stream-spec/blob/master/index.html#L155 And not "instead of". Those are separate features |
Oh, I completely missed that, then my implementations adhere to it by coincidence 😄
Just wondering if this will not be too confusing for people, as there would be two places to set a baseIRI. |
I mostly use the |
As discussed in rdfjs/rdfxml-streaming-parser.js#41, @rubensworks proposed to add a parameter to the import method so that it's a standard feature of all parsers to accept a
baseIRI
This is already implemented by a most and would nicely complement the analogous param on the constructor
The text was updated successfully, but these errors were encountered: