Skip to content
This repository was archived by the owner on Dec 2, 2024. It is now read-only.
/level-jsPublic archive

Support clear() - #182

Merged
vweevers merged 2 commits into
masterfrom
clear
Sep 20, 2019
Merged

Support clear()#182
vweevers merged 2 commits into
masterfrom
clear

Conversation

@vweevers

@vweeversvweevers commented Sep 20, 2019

Copy link
Copy Markdown
Member

Supersedes #175. Ref Level/community#79.

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

@vweeversvweevers added the enhancement New feature or request label Sep 20, 2019
@vweeversvweevers mentioned this pull request Sep 20, 2019
Closed
49 tasks
ralphtheninja
ralphtheninja reviewed Sep 20, 2019
View reviewed changes
Comment threadindex.js
Comment threadutil/clear.js
@vweevers
vweevers merged commit 9a04b3b into masterSep 20, 2019
@vweevers
vweevers deleted the clear branch September 20, 2019 15:00
@vweevers

vweevers commented Sep 20, 2019

Copy link
Copy Markdown
MemberAuthor

Oops, meant to wait for review.

Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@ralphtheninjaralphtheninjaralphtheninja left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

enhancementNew feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vweevers@ralphtheninja