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

Polish clear() #74

Merged
merged 5 commits into from
Sep 12, 2019
Merged

Polish clear() #74

merged 5 commits into from
Sep 12, 2019

Conversation

vweevers
Copy link
Member

@vweevers vweevers commented Sep 8, 2019

Without this PR, clear() works, but risks getting deduped by npm.

I would normally say this (a new feature) is semver-minor but because clear() already floated in via abstract-leveldown as well as levelup, I consider this a fix.

@vweevers vweevers added the semver-patch Bug fixes that are backward compatible label Sep 8, 2019
@vweevers vweevers merged commit b888384 into master Sep 12, 2019
@vweevers vweevers deleted the clear branch September 12, 2019 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-patch Bug fixes that are backward compatible
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants