Skip to content

Commit

Permalink
fix: skip transform delete something.includes (#11530)
Browse files Browse the repository at this point in the history
  • Loading branch information
JLHwung committed May 7, 2020
1 parent 8aa5e57 commit b8a6145
Show file tree
Hide file tree
Showing 7 changed files with 9 additions and 0 deletions.
2 changes: 2 additions & 0 deletions packages/babel-plugin-transform-runtime/src/index.js
Expand Up @@ -358,6 +358,8 @@ export default declare((api, options, dirname) => {
enter(path) {
if (!injectCoreJS) return;
if (!path.isReferenced()) return;
// skip transforming `delete something.includes`
if (path.parentPath.isUnaryExpression({ operator: "delete" })) return;

const { node } = path;
const { object } = node;
Expand Down
Expand Up @@ -86,3 +86,4 @@ object.values(arg);
Function.bind

object.something(arg);
delete object.bind;
Expand Up @@ -240,3 +240,4 @@ _valuesInstanceProperty(object).call(object, arg);
_bindInstanceProperty(Function);

object.something(arg);
delete object.bind;
Expand Up @@ -121,3 +121,4 @@ JSON.parse
Math.pow

Symbol.something
delete Array.from
Expand Up @@ -275,3 +275,4 @@ Date.something;
JSON.parse;
Math.pow;
_Symbol.something;
delete Array.from;
Expand Up @@ -5,3 +5,5 @@ keys(bar).includes;
foo.includes.apply(bar, [1, 2]);

foo.includes = 42;

delete foo.includes;
Expand Up @@ -7,3 +7,4 @@ _includesInstanceProperty(keys(bar));
_includesInstanceProperty(foo).apply(bar, [1, 2]);

foo.includes = 42;
delete foo.includes;

0 comments on commit b8a6145

Please sign in to comment.