8000 gh-104050: Run mypy on `clinic.py` in CI by AlexWaygood · Pull Request #104421 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-104050: Run mypy on clinic.py in CI #104421

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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
May 15, 2023
Merged

gh-104050: Run mypy on clinic.py in CI #104421

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
ca8532b
Slightly annotate cpp.py
erlend-aasland Apr 30, 2023
d2ec2ef
Slightly annotate clinic.py
erlend-aasland May 1, 2023
801af38
Address review
erlend-aasland May 1, 2023
00183ff
Add basic mypy workflow to CI
AlexWaygood May 12, 2023
4bdd409
Make the type check pass
AlexWaygood May 12, 2023
632843b
Tweak workflow file
AlexWaygood May 12, 2023
f7e7a25
improve workflow file
AlexWaygood May 12, 2023
510e08b
Add some more return annotations
AlexWaygood May 12, 2023
773c2ba
simpler glob
AlexWaygood May 12, 2023
6c7ecb9
Address various reviews!
AlexWaygood May 12, 2023
717ee47
more from Hugo
AlexWaygood May 12, 2023
62669d1
Fix ini syntax
AlexWaygood May 12, 2023
ddd0843
Unused import
AlexWaygood May 12, 2023
b7158db
Merge branch 'main' into clinic-mypy-check
AlexWaygood May 12, 2023
9fcd87f 8000
Restore compatibility with Python 3.8
AlexWaygood May 13, 2023
010f0dd
Revert "Restore compatibility with Python 3.8"
AlexWaygood May 15, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Make the type check pass
  • Loading branch information
@AlexWaygood
AlexWaygood committed May 12, 2023
commit 4bdd409fc64f3a584128e0828fad1108f8aa43b4
7 changes: 3 additions & 4 deletions Tools/clinic/clinic.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@

from collections.abc import Callable
from types import *
from typing import NamedTuple, TypeVar, Type
from typing import Any, NamedTuple, Type

# TODO:
#
Expand Down Expand Up @@ -1924,8 +1924,7 @@ def dump(self):
# maps strings to Language objects.
# "languages" maps the name of the language ("C", "Python").
# "extensions" maps the file extension ("c", "py").
L = TypeVar('L', bound=Language)
LangDict = dict[str, L]
LangDict = dict[str, Callable[[str], Language]]

languages = { 'C': CLanguage, 'Python': PythonLanguage }
extensions: LangDict = { name: CLanguage for name in "c cc cpp cxx h hh hpp hxx".split() }
Expand Down Expand Up @@ -2589,7 +2588,7 @@ class CConverter(metaclass=CConverterAutoRegister):

# If not None, default must be isinstance() of this type.
# (You can also specify a tuple of types.)
default_type: Type | tuple[Type, ...] | None = None
default_type: Type[Any] | tuple[Type[Any], ...] | None = None

# "default" converted into a C value, as a string.
# Or None if there is no default.
Expand Down
3 changes: 2 additions & 1 deletion Tools/clinic/cpp.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,8 @@ class Monitor:
Anyway this implementation seems to work well enough for the CPython sources.
"""

is_a_simple_defined: Callable = re.compile(r'^defined\s*\(\s*[A-Za-z0-9_]+\s*\)$').match
is_a_simple_defined: Callable[[str], re.Match[str] | None]
is_a_simple_defined = re.compile(r'^defined\s*\(\s*[A-Za-z0-9_]+\s*\)$').match

def __init__(self, filename=None, *, verbose: bool = False):
self.stack: TokenStack = []
Expand Down
0