-
-
Notifications
You must be signed in to change notification settings - Fork 454
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
feat(transformer): class properties transform #7011
base: 11-04-test_transformer_script_to_amend_babel_fixtures
Are you sure you want to change the base?
feat(transformer): class properties transform #7011
Conversation
Your org has enabled the Graphite merge queue for merging into mainAdd the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
0eb3ad7
to
906e8fd
Compare
CodSpeed Performance ReportMerging #7011 will degrade performances by 14.44%Comparing Summary
Benchmarks breakdown
|
7b2ad83
to
1f4eef9
Compare
366cc8f
to
65400df
Compare
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.
Too much fixtures committed to this repo, let me think about #4048
96a98ba
to
2a57a66
Compare
5663771
to
0c69972
Compare
0c69972
to
3970e03
Compare
Yes, I hope there's a better solution. I just wanted some working tests while I'm building the transform. |
Skeleton of class properties transform. #7011 contains WIP implementation.
c71dda8
to
78b7bed
Compare
63913ed
to
cd8047b
Compare
WIP