Support TypeScript 5.2 using / await using declaration
#15321
Conversation
|
I'll release new patch after this is merged. |
tests/format/js/explicit-resource-management/invalid-for-using-binding-of-of.js
Show resolved
Hide resolved
tests/format/misc/errors/js/explicit-resource-management/invalid-using-binding-let.js
Show resolved
Hide resolved
tests/format/misc/errors/js/explicit-resource-management/invalid-for-using-binding-in.js
Show resolved
Hide resolved
Co-authored-by: fisker Cheung <[email protected]>
There was a problem hiding this comment.
File name. Missing "a"
invalid
^
There was a problem hiding this comment.
This should be valid, I think it's from babel too.
There was a problem hiding this comment.
I believe these two from existing tests? They should be still named "valid".
Better put them back, just let typescript parser to throw.
Co-authored-by: fisker Cheung <[email protected]>
| "valid-for-using-declaration.js", | ||
| "valid-await-using-comments.js", | ||
| "valid-for-lhs-await-as-identifier.js", | ||
| "using-declarations.js", |
There was a problem hiding this comment.
typescript-estree throws for these files (tsc doesn't throw).
|
Size Change: +20.6 kB (0%) Total Size: 8.5 MB
If it's possible, you can run To compare the file with main branch see why the file size grows, and try to remove unnessary code. |
|
@fisker Sure! I'll merge this PR after CI is passed. And I'll try to do reduce bundle size before release. |
|
If you want do it, in my experience, I suggest you do it before merge, it's easier to switch between branches. Just a quick check is enough, maybe obvious unless code can be found, not necessary to dig deep. |
|
I tried a little, but it was difficult, so I gave up once! sorry! |
|
I compared the files, there is noting we can do, we are good. |
2 participants
Description
Fixes #15004
Checklist
docs/directory).changelog_unreleased/*/XXXX.mdfile followingchangelog_unreleased/TEMPLATE.md.✨Try the playground for this PR✨