Skip to content
This repository was archived by the owner on Jul 15, 2018. It is now read-only.

MongoDB 3.X compliant #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

MongoDB 3.X compliant #30

wants to merge 1 commit into from

Conversation

x42en
Copy link

@x42en x42en commented Jun 24, 2018

Hi Oleg,

Here is a simple proposition to comply with recent MongoDB API (3.X).
I'm not sure that I did it the right way, but the node test/test-fibers.js is ok! :D

In order to understand better your work could you explain me in detail what are the steps of the sync() function ? And could you check for exemple that the way I made DB.listCollections() is correct ?

Thank you !

@olegp
Copy link
Owner

olegp commented Jul 15, 2018

Ben, thanks for this and sorry for not getting back to you sooner, been taking some time off.

I had a quick look and there are quite a few changes here which I simply don't have the time to review as I'm no longer using this package myself. This is complicated by the fact that Travis CI seems to be failing all checks associated with this PR.

You're welcome to fork this repo, introduce your changes and publish them to npm under a different name via your own account.

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.

2 participants