Use numeric literals for BYTES_PER_ELEMENT#59567
Use numeric literals for BYTES_PER_ELEMENT#59567lionel-rowe wants to merge 4 commits intomicrosoft:mainfrom
Conversation
|
@typescript-bot test it |
|
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 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! |
There was a problem hiding this comment.
Pull request overview
This PR changes the BYTES_PER_ELEMENT property type from number to specific numeric literal types (1, 2, 4, 8) for all TypedArray types. This enables better type narrowing and allows developers to use BYTES_PER_ELEMENT values in contexts expecting specific numeric literal unions.
Changes:
- Updated TypedArray interface definitions in library files to use numeric literal types instead of
numberforBYTES_PER_ELEMENT - Updated test baselines to reflect the new, more precise error messages that now reference
BYTES_PER_ELEMENTincompatibility instead ofSymbol.toStringTagincompatibility
Reviewed changes
Copilot reviewed 3 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/lib/es5.d.ts | Updated BYTES_PER_ELEMENT to numeric literals (1, 2, 4, 8) for Int8Array, Uint8Array, Uint8ClampedArray, Int16Array, Uint16Array, Int32Array, Uint32Array, Float32Array, and Float64Array |
| src/lib/es2020.bigint.d.ts | Updated BYTES_PER_ELEMENT to literal 8 for BigInt64Array and BigUint64Array |
| src/lib/esnext.float16.d.ts | Updated BYTES_PER_ELEMENT to literal 2 for Float16Array |
| tests/baselines/reference/typedArraysCrossAssignability01.errors.txt | Updated error messages to reflect BYTES_PER_ELEMENT incompatibility instead of Symbol.toStringTag |
| tests/baselines/reference/sharedMemory.types | Updated type annotations to show specific numeric literals for BYTES_PER_ELEMENT |
| tests/baselines/reference/es2024SharedMemory.types | Updated type annotations to show specific numeric literals for BYTES_PER_ELEMENT |
|
@typescript-bot test it |
|
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! |
Fixes #59557