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

Rename the import method #23

Open
martines3000 opened this issue Mar 29, 2024 · 1 comment
Open

Rename the import method #23

martines3000 opened this issue Mar 29, 2024 · 1 comment

Comments

@martines3000
Copy link

martines3000 commented Mar 29, 2024

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: rdfjs/N3.js#380

@elf-pavlik
Copy link
Member

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.

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

No branches or pull requests

2 participants