rdfjs / stream-spec

RDF/JS: Stream interfaces – A specification of a low level interface definition representing RDF data independent of a serialized format in a JavaScript environment.
https://rdf.js.org/stream-spec/
5 stars 2 forks source link

Rename the `import` method #23

Open martines3000 opened 5 months ago

martines3000 commented 5 months ago

Hi.

We are developing a project that is running inside a hardened JavaScript environment called SES (Secure EcmaScript). SES doesn't allow overriding “reserved” keywords like import and this causes many issues for us.

My suggestion is to maybe rename the import() method to importStream(). This way we we avoid using a "reserved" keyword and also make it clearer what the method does. I hope this is not too large of a change and that it can be done. Thanks.

Issue that was opened on the library that implements this spec: https://github.com/rdfjs/N3.js/issues/380

elf-pavlik commented 5 months ago

Since this is a breaking change, it would make sense to consider all other proposed breaking changes and implement them simultaneously. This would require all the implementations to undergo a major release.