Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork3.1k
Update Fix help message url for "None and Optional handling" section#19252
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
A5rocks commentedJun 8, 2025
Maybe it makes more sense to update the docs to support this old anchor, for people on older mypy versions? |
According tomypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅ |
hauntsaninja left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thanks for spotting and fixing! (Open to A5rocks idea in addition!)
f1b496c intopython:masterUh oh!
There was an error while loading.Please reload this page.
Not sure if this will work, since something else has the same anchor on a different page... Seepython#19252, though.
…19828)Addresses#19252 (comment)> support this old anchor, for people on older mypy versionsThis way, when people get the old documentation link, it will continueto go to the right place.I have looked at the documentation that gets generated locally to see ifeverything still works. It does. In our vast panoply of documentation,we now have two very similar refs, `no-strict-optional` and`no_strict_optional` (used inhttps://mypy.readthedocs.io/en/stable/common_issues.html#no-errors-reported-for-obviously-wrong-codeto link tohttps://mypy.readthedocs.io/en/stable/command_line.html#no-strict-optional)but the software handles them fine, without getting confused.
Fixes#19251
Correct url in help message"None and Optional handling" group
Checklist:
Add tests for all changed behaviour.