-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
fix(es/base): Remove unnecessary parameters in helper_expr macro #7296
Conversation
I was thinking something similar. Given that we now process it via a script, I think we can update inline helpers by running a script, too. |
@@ -1,4 +1,4 @@ | |||
var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) { | |||
var _ts_decorate = (this && this._ts_decorate) || function (decorators, target, key, desc) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found that I had to change export name in order to make the ts inline helpers work.
From my personal opinion, I don't particularly like this change. It makes our code look different from the ts helpers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree witg you, but the problem was the SyntaxContext
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
swc-bump:
- swc_ecma_transforms_react
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated review comment generated by auto-rebase script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated review comment generated by auto-rebase script
Description:
We have move to snake_case, so the last parameter of helper_expr is unnecessary.
next step:
We have a copy of the swc/helpers directory in the swc_ecma_transforms_base directory, which has a subtle difference.
It is used for inline helpers.
Currently, it is manually maintained. Can it be generated automatically?
BREAKING CHANGE:
Related issue (if exists):