Introduce ES2025 target & Add missing ScriptTargetFeatures#63046
Introduce ES2025 target & Add missing ScriptTargetFeatures#63046petamoriken wants to merge 14 commits intomicrosoft:mainfrom
Conversation
|
Thanks for the PR! It looks like you've changed the TSServer protocol in some way. Please ensure that any changes here don't break consumers of the current TSServer API. For some extra review, we'll ping @sheetalkamat, @mjbvz, and @joj for you. Feel free to loop in other consumers/maintainers if necessary. |
|
Looks like you're introducing a change to the public API surface area. If this includes breaking changes, please document them on our wiki's API Breaking Changes page. Also, please make sure @DanielRosenwasser and @RyanCavanaugh are aware of the changes, just as a heads up. |
petamoriken
changed the title
|
Can we please get this reviewed? I know the country is burning and everyone is being replaced by AI but this would be really useful for those of us who deal with audiences across different languages. |
|
We have a lot of these lib related PRs; it's absolutely our intention to review all of them as soon as we can, we've just had a bunch of other stuff that has been a bit more pressing. |
|
I will note that an all-up PR for this may not be as useful as us reviewing the individual esnext PRs and then having a dedicated ES2025 PR; them being mixed together in this PR is making it hard to figure out what's new and what's just moving around. I would have preferred that the other PRs not be closed and this PR be a pure move + define options. |
|
Only two files were newly added: es2025.intl.d.ts and es2025.regexp.d.ts. I created this combined PR because the individual "esnext" PRs for these features had stalled, and there had been no progress. I wanted to take over the tasks myself, and since there are only two added files, I thought the review would not be too difficult. @jakebailey If you still prefer to have them separated, I can create two individual PRs instead. What do you think? |
Co-authored-by: lionel-rowe <lionel.rowe@gmail.com>
Co-authored-by: 김상두 <puki4416@gmail.com>
petamoriken
force-pushed
the
feat/es2025-target
branch
from
9d68792 to
86c687a
Compare
|
@jakebailey For now, I can remove the additions of es2025.intl.d.ts and es2025.regexp.d.ts from this PR, but should I do so? Please let me know your thoughts. |
|
@typescript-bot test it |
| { | ||
| "fileInfos": { | ||
| "../../tslibs/ts/lib/lib.es2024.full.d.ts": { | ||
| "../../tslibs/ts/lib/lib.d.ts": { |
There was a problem hiding this comment.
I am not sure why this is changing not to 2025...
|
This PR looks largely correct, though I am confused by some of the test baseline changes. I think it might be fine to take this first and then fix the other lib updates depending on what's in ES2025. |
| ["esnext.array", "lib.esnext.array.d.ts"], | ||
| ["esnext.decorators", "lib.esnext.decorators.d.ts"], | ||
| ["esnext.disposable", "lib.esnext.disposable.d.ts"], | ||
| ["esnext.error", "lib.esnext.error.d.ts"], | ||
| ["esnext.intl", "lib.esnext.intl.d.ts"], | ||
| ["esnext.sharedmemory", "lib.esnext.sharedmemory.d.ts"], | ||
| ["esnext.typedarrays", "lib.esnext.typedarrays.d.ts"], |
There was a problem hiding this comment.
Are none of these applicable to ES2025?
There was a problem hiding this comment.
Yes.
- esnext.array.d.ts
- ES2026 Array.fromAsync
- esnext.decorators.d.ts
- Stage 3 Decorator Metadata
- esnext.disposable.d.ts
- Stage 3 Explicit Resource Management
- esnext.error.d.ts
- ES2026 Error.isError
- esnext.intl.d.ts
- Originally, only the
Intlnamespace was written, and I have not touched this file.
declare namespace Intl { // Empty }
- Originally, only the
- esnext.sharedmemory.d.ts
- Stage 3 Atomics.pause
- esnext.typedarrays.d.ts
- ES2026 Uint8Array to/from Base64
ref: https://github.com/tc39/proposals/blob/main/finished-proposals.md
|
@jakebailey Here are the results of running the user tests with tsc comparing There were infrastructure failures potentially unrelated to your change:
Otherwise... Everything looks good! |
|
Hey @jakebailey, the results of running the DT tests are ready. Everything looks the same! |
|
@jakebailey Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@jakebailey Here are the results of running the top 400 repos with tsc comparing Everything looks good! |
4 participants
Fixes #61735
This PR includes the addition of 2 type files:
Related issues and (stalled) PRs
ES2025
RegExp.escape(es2025.regexp.d.ts)Closes #61321
Closes #62138
ES2025
Intl.DurationFormat(es2025.intl.d.ts)Closes #60608
Closes #60646