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

Jump to bottom

Add Storage::range_keys and Storage::range_values #1748

Merged
merged 1 commit into from
Jun 27, 2023
Merged

Add Storage::range_keys and Storage::range_values #1748

merged 1 commit into from
Jun 27, 2023

Conversation

webmaster128
Copy link
Member

@webmaster128 webmaster128 commented Jun 27, 2023 •
edited
Loading

This is a first non-breaking change for #1498

@webmaster128 webmaster128 requested a review from chipshort June 27, 2023 06:28
chipshort
chipshort approved these changes Jun 27, 2023
View reviewed changes
Copy link
Collaborator

@chipshort chipshort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Nice that we can add this in a non-breaking way

@webmaster128
Copy link
Member Author

Thanks! Let me rebase this to 1.3 then.

@webmaster128 webmaster128 self-assigned this Jun 27, 2023
@webmaster128 webmaster128 added this to the 1.3.0 milestone Jun 27, 2023
@webmaster128 webmaster128 force-pushed the add-range_keys-range_values branch from ef3d0e8 to a9d2b0a Compare June 27, 2023 13:22
@webmaster128 webmaster128 changed the base branch from main to release/1.3 June 27, 2023 13:27
@webmaster128 webmaster128 merged commit ebc5773 into release/1.3 Jun 27, 2023
@webmaster128 webmaster128 deleted the add-range_keys-range_values branch June 27, 2023 13:48
@webmaster128 webmaster128 mentioned this pull request Aug 17, 2023
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@chipshort chipshort chipshort approved these changes

Assignees

@webmaster128 webmaster128

Labels
better-iterators
Projects
None yet
Milestone
1.3.0
Development

Successfully merging this pull request may close these issues.
2 participants
@webmaster128 @chipshort