Skip to content
This repository has been archived by the owner on Dec 1, 2024. It is now read-only.

abstract-leveldown ~2.1.0 #143

Merged
merged 1 commit into from
Mar 21, 2015
Merged

abstract-leveldown ~2.1.0 #143

merged 1 commit into from
Mar 21, 2015

Conversation

ralphtheninja
Copy link
Member

No description provided.

@rvagg
Copy link
Member

rvagg commented Jan 19, 2015

@ralphtheninja do tests all still pass with this?

@ralphtheninja
Copy link
Member Author

Yep! I have tested with io.js v1.0.2, v1.0.3 and node v0.10.35.

$ t

> [email protected] test /home/lms/src/node-leveldown
> tap test/*-test.js --stderr

ok test/approximate-size-test.js ...................... 16/16
ok test/batch-test.js ................................. 37/37
ok test/chained-batch-test.js ......................... 19/19
ok test/cleanup-hanging-iterators-test.js ............. 20/20
ok test/close-test.js ................................... 7/7
ok test/del-test.js ................................... 12/12
ok test/destroy-test.js ............................... 17/17
ok test/get-test.js ................................... 69/69
ok test/getproperty-test.js ........................... 16/16
ok test/iterator-recursion-test.js ...................... 4/4
ok test/iterator-test.js ............................ 136/136
ok test/leveldown-test.js ............................... 5/5
ok test/make-test.js .................................... 1/1
ok test/open-test.js .................................. 16/16
ok test/port-libuv-fix-test.js .......................... 3/3
ok test/put-get-del-test.js ......................... 144/144
ok test/put-test.js ................................... 10/10
ok test/ranges-test.js .............................. 129/129
ok test/repair-test.js ................................ 11/11
total ............................................... 672/672

@juliangruber
Copy link
Member

lgtm then

@ralphtheninja
Copy link
Member Author

I'm merging this today. If there is any project that should follow abstract-leveldown it's this project ;)

@ralphtheninja
Copy link
Member Author

Ok. Merging. I've also tested it on 0.12 and [email protected].

ralphtheninja added a commit that referenced this pull request Mar 21, 2015
@ralphtheninja ralphtheninja merged commit 4919697 into Level:master Mar 21, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants