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

Have Iterator implement Enumerable #41

Closed
AndrewSisley opened this issue Jan 28, 2025 · 0 comments · Fixed by #45
Closed

Have Iterator implement Enumerable #41

AndrewSisley opened this issue Jan 28, 2025 · 0 comments · Fixed by #45
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@AndrewSisley
Copy link
Collaborator

Remove the Valid function. Have Seek, Next, and maybe store.Iter return booleans. Add a Reset function.

Strongly consider not directly embedding Enumerable within the Iterator interface, but instead heavily document and test that it implements it - this will save a production code reference to the Enumerable package.

This change will facilitate #23 but do not go out of your way to include that within this work-item.

@AndrewSisley AndrewSisley added the enhancement New feature or request label Jan 28, 2025
@AndrewSisley AndrewSisley added this to the CoreKV v0.1 milestone Jan 28, 2025
@AndrewSisley AndrewSisley self-assigned this Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant