FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Simplify stub for `datetime.datetime` by AlexWaygood · Pull Request #6320 · python/typeshed · GitHub

Repository navigation

Simplify stub for datetime.datetime - #6320

Merged
srittau merged 2 commits into
python:masterfrom
AlexWaygood:simplify-datetime
Nov 16, 2021
Merged

srittau merged 2 commits into
python:masterfrom
AlexWaygood:simplify-datetime

Conversation

Copy link
Copy Markdown
Member

The stub for datetime.datetime unnecessarily includes many methods that have identical signatures to the methods in its parent class, datetime.date. This is unnecessary.

(All of my proposed deletions are methods that are directly inherited from datetime.date, except for __hash__ and timetuple. These two are redefined in datetime.date, but maintain the same signature in the subclass that they have in the superclass.)

This comment has been minimized.

1 similar comment

Copy link
Copy Markdown
Contributor

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

srittau merged commit 7598396 into python:master Nov 16, 2021
AlexWaygood deleted the simplify-datetime branch November 16, 2021 18:05
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL