Skip to content

refactor(transformer/class-properties): move transform logic of callee of CallExpression to transform_private_field_callee#7503

Merged
graphite-app[bot] merged 1 commit intomainfrom
11-27-refactor_transformer_class-properties_move_transform_logic_of_callee_of_callexpression_to_transform_private_field_callee
Nov 27, 2024
Merged

refactor(transformer/class-properties): move transform logic of callee of CallExpression to transform_private_field_callee#7503
graphite-app[bot] merged 1 commit intomainfrom
11-27-refactor_transformer_class-properties_move_transform_logic_of_callee_of_callexpression_to_transform_private_field_callee

Conversation

@Dunqing
Copy link
Member

@Dunqing Dunqing commented Nov 27, 2024

This transform logic is also can be used to tag of TaggedTemplateExpression, so let's move it to a separate function.

@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 27, 2024

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.

@github-actions github-actions bot added the A-transformer Area - Transformer / Transpiler label Nov 27, 2024
@Dunqing Dunqing force-pushed the 11-27-refactor_transformer_class-properties_move_transform_logic_of_callee_of_callexpression_to_transform_private_field_callee branch from 49cc2a4 to 96356dd Compare November 27, 2024 09:26
@Dunqing Dunqing changed the title refactor(transformer/class-properties) move transform logic of callee of CallExpression to transform_private_field_callee refactor(transformer/class-properties): move transform logic of callee of CallExpression to transform_private_field_callee Nov 27, 2024
@github-actions github-actions bot added the C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior label Nov 27, 2024
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 27, 2024

CodSpeed Performance Report

Merging #7503 will not alter performance

Comparing 11-27-refactor_transformer_class-properties_move_transform_logic_of_callee_of_callexpression_to_transform_private_field_callee (968863b) with main (5261547)

Summary

✅ 30 untouched benchmarks

@overlookmotel overlookmotel force-pushed the 11-27-refactor_transformer_class-properties_move_transform_logic_of_callee_of_callexpression_to_transform_private_field_callee branch 2 times, most recently from efd6f3a to f11f2a6 Compare November 27, 2024 10:37
Copy link
Member

@overlookmotel overlookmotel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've pushed a commit to include output for static props in comments.

@Dunqing If you're happy with that, please merge.

@graphite-app graphite-app bot added the 0-merge Merge with Graphite Merge Queue label Nov 27, 2024
@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 27, 2024

Merge activity

  • Nov 27, 6:25 AM EST: A user added this pull request to the Graphite merge queue.
  • Nov 27, 6:30 AM EST: The Graphite merge queue couldn't merge this PR because it had conflicts with the trunk branch.
  • Nov 27, 7:04 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
  • Nov 27, 7:07 AM EST: A user added this pull request to the Graphite merge queue.
  • Nov 27, 7:13 AM EST: A user merged this pull request with the Graphite merge queue.

@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Nov 27, 2024
@overlookmotel
Copy link
Member

I caused merge conflicts with one of my other PRs. I'll fix and merge.

@overlookmotel overlookmotel force-pushed the 11-27-refactor_transformer_class-properties_move_transform_logic_of_callee_of_callexpression_to_transform_private_field_callee branch from f11f2a6 to 80b4cff Compare November 27, 2024 12:02
@overlookmotel overlookmotel added the 0-merge Merge with Graphite Merge Queue label Nov 27, 2024
…ee` of `CallExpression` to `transform_private_field_callee` (#7503)

This transform logic is also can be used to `tag` of `TaggedTemplateExpression`, so let's move it to a separate function.
@overlookmotel overlookmotel force-pushed the 11-27-refactor_transformer_class-properties_move_transform_logic_of_callee_of_callexpression_to_transform_private_field_callee branch from 80b4cff to 968863b Compare November 27, 2024 12:08
@graphite-app graphite-app bot merged commit 968863b into main Nov 27, 2024
@graphite-app graphite-app bot deleted the 11-27-refactor_transformer_class-properties_move_transform_logic_of_callee_of_callexpression_to_transform_private_field_callee branch November 27, 2024 12:13
@oxc-bot oxc-bot mentioned this pull request Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0-merge Merge with Graphite Merge Queue A-transformer Area - Transformer / Transpiler C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants