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.

Sign up for GitHub

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

Jump to bottom

[2.0] Remove slaveOkay #1673

Closed
wants to merge 16 commits into from
Closed

[2.0] Remove slaveOkay #1673

wants to merge 16 commits into from

Conversation

malarzm
Copy link
Member

@malarzm malarzm commented Nov 14, 2017

This PR removes deprecated slaveOkay.

@malarzm malarzm added the Task label Nov 14, 2017
@malarzm malarzm added this to the 2.0.0 milestone Nov 14, 2017
@malarzm malarzm requested a review from alcaeus November 14, 2017 21:18
@malarzm
Copy link
Member Author

malarzm commented Nov 14, 2017

@alcaeus this should actually be incorporated into #1553 now I look into that again :)

@alcaeus
Copy link
Member

alcaeus commented Nov 15, 2017

I’ll take a look and merge it into develop or the driver branch later today. I remember doing a bunch of slaveOkay cleanup when switching to ext-mongodb.

@alcaeus alcaeus self-assigned this Nov 15, 2017
@alcaeus alcaeus closed this Dec 22, 2017
@malarzm malarzm mentioned this pull request Jan 7, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alcaeus alcaeus Awaiting requested review from alcaeus

Assignees

@alcaeus alcaeus

Labels
Task
Projects
None yet
Milestone
2.0.0-Beta1
Development

Successfully merging this pull request may close these issues.
3 participants
@malarzm @alcaeus @coudenysj