-
Notifications
You must be signed in to change notification settings - Fork 13.2k
Return iterable of RegExpExecArray from RegExp#[Symbol.matchAll] #60516
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
Conversation
2fe6cd8 to
313b9d4
Compare
313b9d4 to
11944f3
Compare
|
@typescript-bot test it |
|
@jakebailey Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Hey @jakebailey, the results of running the DT tests are ready. Everything looks the same! |
|
@jakebailey Here are the results of running the user tests with tsc comparing Everything looks good! |
|
@jakebailey Here are the results of running the top 400 repos with tsc comparing Everything looks good! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Aligns TypeScript’s lib typing for RegExp#[Symbol.matchAll] with the already-correct String#matchAll behavior by returning an iterator of RegExpExecArray (fixing #60515).
Changes:
- Updated
RegExp#[Symbol.matchAll]to returnRegExpStringIterator<RegExpExecArray>instead ofRegExpMatchArray. - Updated compiler test baselines to reflect the new iterator element type (including
index/inputbecoming non-optional).
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| tests/baselines/reference/regexMatchAll.types | Baseline updated to show RegExpExecArray element type and required index/input. |
| tests/baselines/reference/regexMatchAll-esnext.types | Same baseline update for the esnext target test. |
| src/lib/es2020.symbol.wellknown.d.ts | Changes RegExp#[Symbol.matchAll] return type to RegExpStringIterator<RegExpExecArray>. |
Fixes #60515