-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Avoid __proto__ syntax #3140
Comments
What are the trade-offs between the approaches?
|
Approach 1 is recommended against in the specification. Do we want to change this ? If yes I could submit a PR, otherwise feel free to close the issue. |
I cannot find any specific recommendation in the document you linked. As a matter of fact, this document properly defines the semantics of accessing Also it is very concise syntactically. The Thus our approach is the only one I would feel good to have enabled by default, and It has already been discussed with various experts. If there are concerns about the optional nature of the feature, we would be open to a PR that adds an option to deactivate it, similar to |
Expected Behavior / Situation
rollup does not use deprecated syntax.
Actual Behavior / Situation
rollup uses the deprecated
__proto__
syntax https://www.ecma-international.org/ecma-262/10.0/index.html#sec-additional-ecmascript-features-for-web-browsersJust saw #3136
Modification Proposal
Replace
with
or
or
The text was updated successfully, but these errors were encountered: