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

Allow collection options to be passed only via kwargs #240

Merged

Conversation

stevenkaras
Copy link
Contributor

This is in line with the pymongo behavior, and will make it easier to migrate code from pymongo to txmongo.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 94.331% when pulling e54e5c0 on cogitosys:bugfixes/collection-creation-options into 06b89d2 on twisted:master.

4 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 94.331% when pulling e54e5c0 on cogitosys:bugfixes/collection-creation-options into 06b89d2 on twisted:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 94.331% when pulling e54e5c0 on cogitosys:bugfixes/collection-creation-options into 06b89d2 on twisted:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 94.331% when pulling e54e5c0 on cogitosys:bugfixes/collection-creation-options into 06b89d2 on twisted:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 94.331% when pulling e54e5c0 on cogitosys:bugfixes/collection-creation-options into 06b89d2 on twisted:master.

@coveralls
Copy link

coveralls commented Oct 31, 2018

Coverage Status

Coverage decreased (-0.04%) to 94.331% when pulling f0b257f on cogitosys:bugfixes/collection-creation-options into 06b89d2 on twisted:master.

@IlyaSkriblovsky
Copy link
Contributor

I think this is the very good idea to be more in line with pymongo. Thanks!
/cc @psi29a

@psi29a
Copy link
Contributor

psi29a commented Nov 1, 2018

Agreed!

@psi29a
Copy link
Contributor

psi29a commented Nov 1, 2018

@stevenkaras can you add an entry here:
https://github.com/twisted/txmongo/blob/master/docs/source/NEWS.rst
Something like:

Release 18.3.0 (UNRELEASED)

  • Feature:

Once we have that, we'll merge! :)

Thank you very much1

This is in line with the pymongo behavior[0], and will make it easier to
migrate code from pymongo to txmongo.

[0]: https://github.com/mongodb/mongo-python-driver/blob/3.7.2/pymongo/database.py#L304
@stevenkaras stevenkaras force-pushed the bugfixes/collection-creation-options branch from e54e5c0 to f0b257f Compare November 1, 2018 13:02
@stevenkaras
Copy link
Contributor Author

@psi29a is there anything else?

@psi29a
Copy link
Contributor

psi29a commented Nov 21, 2018

Good to go! Thanks. :)

@psi29a psi29a merged commit b3357d6 into twisted:master Nov 21, 2018
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.

None yet

4 participants