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

Add missing default arguments for AsyncIOMotorClient #1

Merged
merged 1 commit into from
Feb 12, 2023
Merged

Add missing default arguments for AsyncIOMotorClient #1

merged 1 commit into from
Feb 12, 2023

Conversation

lmichaelis
Copy link
Contributor

Hi there,
this commit adds default arguments for the following member functions of AsyncIOMotorClient:

  • get_database,
  • get_default_database,
  • list_databases and
  • start_session

The original source code containing these defaults can be found at pymongo/mongo_client.py.

This commit adds default arguments for the following member functions of `AsyncIOMotorClient`:
* `get_database`,
* `get_default_database`,
* `list_databases` and
* `start_session`

The original source code containing these defaults can be found at [pymongo/mongo_client.py](https://github.com/mongodb/mongo-python-driver/blob/master/pymongo/mongo_client.py).
@L0RD-ZER0
Copy link
Owner

I apologise for the late reply, I'll check the PR and get back to you shortly. Thanks for your contribution!

Copy link
Owner

@L0RD-ZER0 L0RD-ZER0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for finding out that these argument defaults were missing. I'll merge the commit.

@L0RD-ZER0 L0RD-ZER0 merged commit a89c46c into L0RD-ZER0:master Feb 12, 2023
@L0RD-ZER0 L0RD-ZER0 added the good first issue Good for newcomers label Feb 12, 2023
L0RD-ZER0 added a commit that referenced this pull request Feb 12, 2023
Added missing optional arguments' default values as indicated in #1.
@L0RD-ZER0
Copy link
Owner

As of 2491fa8, there shouldn't be any other missing optional type-hints. The package on PyPI has been updated and changes should reflect on update. Once again, thanks for finding this one out!

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

Successfully merging this pull request may close these issues.

None yet

2 participants