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.

Sign up for GitHub

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

Jump to bottom

Support clear() #182

Merged
merged 2 commits into from
Sep 20, 2019
Merged

Support clear() #182

merged 2 commits into from
Sep 20, 2019

Conversation

vweevers
Copy link
Member

Supersedes #175. Ref Level/community#79.

The default _clear() of abstract-leveldown doesn't work here, because some environments lack binary key support.

@vweevers vweevers added the enhancement New feature or request label Sep 20, 2019
@vweevers vweevers mentioned this pull request Sep 20, 2019
Closed
49 tasks
ralphtheninja
ralphtheninja reviewed Sep 20, 2019
View reviewed changes
index.js Show resolved Hide resolved
util/clear.js Show resolved Hide resolved
@vweevers vweevers merged commit 9a04b3b into master Sep 20, 2019
@vweevers vweevers deleted the clear branch September 20, 2019 15:00
@vweevers
Copy link
Member Author

Oops, meant to wait for review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ralphtheninja ralphtheninja ralphtheninja left review comments

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vweevers @ralphtheninja