Level / level-test

Inject temporary and isolated level stores (leveldown, level-js, memdown or custom) into your tests.
MIT License
20 stars 12 forks source link

Remove clean option #113

Closed vweevers closed 5 years ago

vweevers commented 5 years ago

Because 1) it should not be needed if the location is unique 2) it's a noop in node and 3) doesn't even work properly in browsers:

https://github.com/Level/level-test/blob/bba85479ae3595e4b88932acfa3ca632f42ab3cf/browser.js#L16-L19

vweevers commented 5 years ago

Hmm, could removing something that doesn't work be considered semver-patch?

Edit: Doesn't matter, next release is a major anyway for dropping node 6.