refactor(transformer/tagged-template): reorder functions#18032
Conversation
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
CodSpeed Performance ReportMerging this PR will not alter performanceComparing Summary
Footnotes
|
There was a problem hiding this comment.
Pull request overview
This PR refactors the tagged_template_transform.rs file by reordering functions to place the main entry point at the top of the implementation. The transform_tagged_template method is moved from its position after helper functions to immediately after the constructor, improving code readability and organization.
Changes:
- Moved
transform_tagged_templatemethod to be the first method after the constructor - Added a period to the doc comment for consistency
- Added a
debug_assert!to verify the expression type precondition
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Merge activity
|
Follow-on after #15834. Pure refactor. Re-order functions so the main entry to the transform is at the top. Also add a `debug_assert!`.
5c32920 to
b332934
Compare
4aede96 to
b02ad91
Compare

Follow-on after #15834. Pure refactor. Re-order functions so the main entry to the transform is at the top.
Also add a
debug_assert!.