-
-
Notifications
You must be signed in to change notification settings - Fork 708
fix(transformer): convert enum numbers to strings in template literals #15183
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
base: main
Are you sure you want to change the base?
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. |
Co-authored-by: Dunqing <[email protected]>
Co-authored-by: Dunqing <[email protected]>
CodSpeed Performance ReportMerging #15183 will not alter performanceComparing Summary
Footnotes
|
|
@copilot CI job fails, the reason is that you have to update the coverage snapshots |
Co-authored-by: Dunqing <[email protected]>
Summary
Fixed a bug where numeric enum values in TypeScript were not converted to strings when used in template literals. The fix converts
ConstantValue::Numberto strings usingto_js_string()method, matching how binary addition already handles number-to-string conversion for template literals.Updated the transform conformance snapshot to include the new test case.
Security Summary
No security vulnerabilities were found in the changes.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.