Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Propagate getRange support in Listable when available #252

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

manzt
Copy link
Owner

@manzt manzt commented Feb 24, 2025

Previously, only Readable.get was propagated up. This makes sure we can also
use Readable.getRange if supported when the store is consolidated.

Copy link

changeset-bot bot commented Feb 24, 2025

🦋 Changeset detected

Latest commit: eb68e0d

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

This PR includes changesets to release 4 packages
Name Type
@zarrita/core Patch
@zarrita/indexing Patch
@zarrita/ndarray 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 added the enhancement New feature or request label Feb 24, 2025
@manzt manzt force-pushed the manzt/conslidated-forward-get-range branch 2 times, most recently from 3571f93 to b7d70a6 Compare February 24, 2025 17:51
Previously, only `get` was propagated up. This makes sure we can also
use `getRange` if supported when the store is consolidated.
@manzt manzt force-pushed the manzt/conslidated-forward-get-range branch from b7d70a6 to eb68e0d Compare February 24, 2025 17:51
@manzt manzt mentioned this pull request Feb 24, 2025
@manzt manzt merged commit 6151346 into main Feb 24, 2025
2 checks passed
@manzt manzt deleted the manzt/conslidated-forward-get-range branch February 24, 2025 17:52
@github-actions github-actions bot mentioned this pull request Feb 24, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant