Contributor

• TelegramObject.__getitem__ now raises KeyError instead of returning None, and a special case is made for obj['from'] to look up obj['from_user'].
• Removed __getitem__ override from Message
• Test for Message changed to check for KeyError instead of None.

Checklist for PRs

• Added .. versionadded:: version, .. versionchanged:: version or .. deprecated:: version to the docstrings for user facing changes (for methods/class descriptions, arguments and attributes)
• Created new or adapted existing unit tests
• Added myself alphabetically to AUTHORS.rst (optional)

Member

Choose a reason for hiding this comment

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

Member

There's a pre-commit test failing. Can you add a pylint ignore on that line? see the other places in the code on how to add a pylint ignore.

rather just do what pylint suggests to do ;)

Contributor Author

Addressed your comments 😄

Member

Choose a reason for hiding this comment

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

Contributor Author

A lot of things came up at school and job. If someone wants to clean up and finish, feel free to do so :)