-
Notifications
You must be signed in to change notification settings - Fork 13k
Add missing Float16Array constructors #62351
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
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
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
Float16Array
constructor overloads to match existing patterns for other typed arrays - Updates test coverage to include
Float16Array
in 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`
9a46df3
to
97664b0
Compare
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! |
These 2 constructor overloads appear to be missing. You can find similar overloads for other typed array constructors in
es6.d.ts
Fixes #62343