Add missing Float16Array constructors#62351
Add missing Float16Array constructors#62351jakebailey merged 1 commit intomicrosoft:mainfrom greggman:float16-issue
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds missing constructor overloads for the Float16Array type to bring it in line with other typed array constructors. The change ensures type consistency across all typed array types in TypeScript's standard library definitions.
- Adds
Float16Arrayconstructor overloads to match existing patterns for other typed arrays - Updates test coverage to include
Float16Arrayin typed array constructor validation
|
@microsoft-github-policy-service agree |
|
This LGTM but needs a format to be accepted. |
These 2 constructor overloads appear to be missing. You can find similar overloads for other typed array constructors in `es6.d.ts`
|
ran |
|
@typescript-bot test it probably won't say anything interesting, but |
|
@typescript-bot cherry-pick this to release-5.9 |
|
Hey, @jakebailey! I've created #62426 for you. |
|
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 There were infrastructure failures potentially unrelated to your change:
Otherwise... Everything looks good! |
|
@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! |
…e-5.9 (#62426) Co-authored-by: Greggman <[email protected]>
These 2 constructor overloads appear to be missing. You can find similar overloads for other typed array constructors in
es6.d.tsFixes #62343