Skip to content

Commit

Permalink
address review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
JLHwung committed Jul 11, 2020
1 parent b7f9cf5 commit 1e796b3
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 9 deletions.
2 changes: 1 addition & 1 deletion packages/babel-plugin-transform-parameters/src/params.js
Expand Up @@ -229,6 +229,6 @@ function buildScopeIIFE(shadowedParams, body) {
}

return t.returnStatement(
t.callExpression(t.arrowFunctionExpression(args, body), params),
t.callExpression(t.arrowFunctionExpression(params, body), args),
);
}
5 changes: 2 additions & 3 deletions packages/babel-plugin-transform-typescript/src/index.js
Expand Up @@ -134,9 +134,8 @@ export default declare(
);
}

return template.statement.ast`this.${t.cloneNode(
id,
)} = ${t.cloneNode(id)}`;
return template.statement.ast`
this.${t.cloneNode(id)} = ${t.cloneNode(id)}`;
});

injectInitialization(classPath, path, assigns);
Expand Down
8 changes: 3 additions & 5 deletions packages/babel-plugin-transform-typescript/src/namespace.js
Expand Up @@ -156,12 +156,10 @@ function handleNested(path, t, node, parentExport) {
let fallthroughValue = t.objectExpression([]);

if (parentExport) {
const memberExpr = t.memberExpression(parentExport, realName);
fallthroughValue = template.expression.ast`
${parentExport}.${t.cloneNode(realName)} || (
${t.cloneNode(parentExport)}.${t.cloneNode(
realName,
)} = ${fallthroughValue}
)
${t.cloneNode(memberExpr)} ||
(${t.cloneNode(memberExpr)} = ${fallthroughValue})
`;
}

Expand Down

0 comments on commit 1e796b3

Please sign in to comment.