Skip to content

Conversation

@oschwald
Copy link
Member

@oschwald oschwald commented Jul 9, 2020

No description provided.

@coveralls
Copy link

coveralls commented Jul 9, 2020

Coverage Status

Coverage decreased (-2.4%) to 93.491% when pulling 38ca1f7 on greg/py3 into a4b31a3 on master.

Copy link
Contributor

@horgh horgh left a comment

Choose a reason for hiding this comment

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

Cool, some nice improvements and simplifications with this!

maxminddb.types
~~~~~~~~~~~~~~~
This module provides a Record type the represents a database record.
Copy link
Contributor

Choose a reason for hiding this comment

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

"that represents" perhaps?

) -> Reader:
database: Union[AnyStr, int, os.PathLike, IO], mode: int = MODE_AUTO
) -> Union[Reader, "maxminddb.extension.Reader"]:
"""Open a Maxmind DB database
Copy link
Contributor

Choose a reason for hiding this comment

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

"MaxMind"?

@horgh horgh merged commit e633b9a into master Jul 10, 2020
@horgh horgh deleted the greg/py3 branch July 10, 2020 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants