Skip to content
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

Fix TypedArray types for TypeScript < 5.7 #239

Merged
merged 1 commit into from
Dec 3, 2024
Merged

Conversation

manzt
Copy link
Owner

@manzt manzt commented Dec 3, 2024

TypeScript changed the typing behavior of all TypedArray objects to
now be generic over the underlying ArrayBufferLike type. In order to
have our types be consistent with these changes, we needed to specify
the ArrayBuffer explicitly, but that breaks for older versions of
TypeScript. This PR fixes the version of TypeScript to 5.6. We will need
to bump again when we want to support 5.7.

Copy link

changeset-bot bot commented Dec 3, 2024

🦋 Changeset detected

Latest commit: 47dedc6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@zarrita/core Patch
@zarrita/indexing Patch
@zarrita/ndarray Patch
@zarrita/storage Patch
@zarrita/typedarray Patch
zarrita Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@manzt manzt force-pushed the manzt/typescript-5.6 branch from f2d4bd2 to 56360c6 Compare December 3, 2024 22:09
TypeScript changed the typing behavior of all `TypedArray` objects to
now be generic over the underlying `ArrayBufferLike` type. In order to
have our types be consistent with these changes, we needed to specify
the `ArrayBuffer` explicitly, but that breaks for older versions of
TypeScript. This PR fixes the version of TypeScript to 5.6. We will need
to bump again when we want to support 5.7.
@manzt manzt force-pushed the manzt/typescript-5.6 branch from 56360c6 to 47dedc6 Compare December 3, 2024 22:10
@manzt manzt merged commit 7756cdc into main Dec 3, 2024
2 checks passed
@manzt manzt deleted the manzt/typescript-5.6 branch December 3, 2024 22:11
@github-actions github-actions bot mentioned this pull request Dec 3, 2024
@github-actions github-actions bot mentioned this pull request Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant