refactor(transformer/class-properties): remove pointless method

[overlookmotel]

#7516 removed the need for a special method to handle export default class {}. Delete it.

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add 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.

[overlookmotel]

• refactor(transformer/class-properties): remove pointless method #7592 👈 (View in Graphite)
• fix(transformer/class-properties): create temp var for class where required #7516
• refactor(transformer/class-properties): PrivatePropsStack type #7589
• docs(transformer/class-properties): add missing docs #7588
• refactor(transformer/class-properties): move creating temp var out of main loop #7587
• refactor(transformer/class-properties): exit transform_class faster if nothing to do #7586
• refactor(transformer/class-properties): ResolvedPrivateProp type #7532
• refactor(transformer/class-properties): shortcut_static_class take SymbolId #7531
• refactor(transformer/class-properties): rename class_binding #7533
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codspeed-hq]

CodSpeed Performance Report

Merging #7592 will not alter performance

_{Comparing 12-03-refactor_transformer_class-properties_remove_pointless_method (802233d) with 11-27-fix_transformer_class-properties_create_temp_var_for_class_where_required (0eadd9f)}

Summary

✅ 30 untouched benchmarks

[overlookmotel]

@Dunqing Getting this working has had me tearing my hair out for days. I suggest not giving it a full review - I'm as sure as I can be that it's correct now, and it'll likely only have you tearing your hair out too!

But please do take a look at the changes to transform_private_field_expression_impl in #7532 and #7516, as I've had to change the APIs related to private fields, which will affect your work on this?.#prop. I don't think it'll make a dramatic difference, but your PR will need a few tweaks to align with the changes here. Happy to help if it causes you trouble.

[Dunqing]

Merge activity

• Dec 3, 2:44 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Dec 3, 2:44 AM EST: A user added this pull request to the Graphite merge queue.
• Dec 3, 3:44 AM EST: A user merged this pull request with the Graphite merge queue.