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

Update mongoskin to v2 #119

Closed
wants to merge 1 commit into from
Closed

Conversation

ratson
Copy link
Contributor

@ratson ratson commented Dec 20, 2015

I have upgraded mongoskin to v2 and fixed the code to pass the tests.
This should fix #93 and #111

For those who can't wait, feel free to try my fork and report issues.

@travisjeffery Would you review the code and see if anything need to be improved?

@YasharF
Copy link

YasharF commented Jan 23, 2016

Any updates on merging this pull request?

@vnglst
Copy link

vnglst commented Mar 8, 2016

Any updates on merging this PR?

@LeNiglo
Copy link

LeNiglo commented Apr 10, 2016

Is there a way that this PR is merged ?
There is apparently no conflicts, so ... what's the matter ?

@ratson
Copy link
Contributor Author

ratson commented Apr 10, 2016

@LeNiglo Feel free to try out my fork, monkii. I really don't see this will be merged soon as the author has not been replying any issues for quite a while.

@LeNiglo
Copy link

LeNiglo commented Apr 10, 2016

I'm already using it @ratson. But still ...

@YasharF
Copy link

YasharF commented Apr 11, 2016

I just also emailed @travisjeffery to see if he can help out or if he can not, to find out who the right contact would be.

@YasharF
Copy link

YasharF commented Apr 11, 2016

By the way, the PR, is setting the dependencies to:

"mongodb": "^2.1",
"mongoskin": "~2.0.3",
"mpromise": "0.5.1"

However, I think there are newer versions for them at the moment. Can you please update the versions?

mongoskin: 2.1.0 is the latest
mpromise: 0.5.5 is the latest

@ratson
Copy link
Contributor Author

ratson commented Apr 15, 2016

@YasharF I can not update this pull request as monkii is no longer a fork, here is the updated pull request.

@ratson ratson closed this Apr 15, 2016
@YasharF
Copy link

YasharF commented Apr 15, 2016

Is the intent of Monkii a replacement of Monk and calling Monk as an abandoned project? If so, there are other pending pull requests and issues filed for Monk, are you planning to address them in Monkii?

@ratson
Copy link
Contributor Author

ratson commented Apr 16, 2016

I really wish there is monk v2, which will be a rewrite without mongoskin. Unfortunately, it is not happening.

monkii serve mainly for upgrading mongoskin and allow usage of the latest MongoDB driver. I am open to accept any bug fixes, or additional features that do not introduce breaking changes.

@YasharF
Copy link

YasharF commented Apr 23, 2016

I emailed Guillermo Rauch and also sent a message to Automattic Inc. to see if we can revive this package. If not, we may just need to fork it, and blast the internet that monk is dead and our new replacement is .... Please update your site, guide, etc and have them use our new package instead of Monk.

@rauchg
Copy link
Contributor

rauchg commented May 17, 2016

Unfortunately I'm not with Automattic anymore @YasharF! I don't have push access to this :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

MongoDB 2.0 Driver
5 participants