Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

gh-108455: peg_generator: make the mypy config slightly stricter#108627

Merged
AlexWaygood merged 4 commits into
python:mainpython/cpython:mainfrom
AlexWaygood:peg-mypy-stricterAlexWaygood/cpython:peg-mypy-stricterCopy head branch name to clipboard
Aug 29, 2023
Merged

gh-108455: peg_generator: make the mypy config slightly stricter#108627
AlexWaygood merged 4 commits into
python:mainpython/cpython:mainfrom
AlexWaygood:peg-mypy-stricterAlexWaygood/cpython:peg-mypy-stricterCopy head branch name to clipboard

Conversation

@AlexWaygood

@AlexWaygood AlexWaygood commented Aug 29, 2023

Copy link
Copy Markdown
Member

This picks some low-hanging fruit when it comes to the mypy config, enabling various stricter settings that don't require large code changes:

  • Enable --no-explicit-reexport
  • Enable the truthy-bool and ignore-without-code error codes
  • Explicitly note in our config file that --warn-unreachable can't yet be enabled. (It would take several code changes before we could enable it; it's deferred for now.)

@AlexWaygood AlexWaygood enabled auto-merge (squash) August 29, 2023 11:19
@AlexWaygood AlexWaygood merged commit 0d140b8 into python:main Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.