Closed Avasam closed 2 weeks ago
Updated with latest review comments! Btw feel free to squash commit once this is accepted. I avoid rebases/force-pushes on GitHub to make it easier on the reviewer to see latest changes.
What is the status of this PR?
What is the status of this PR?
Unless you have any other comment, it's good to be squash-merged on my side if it passes the CI. https://github.com/microsoft/python-type-stubs/pull/301#discussion_r1714086800 and https://github.com/microsoft/python-type-stubs/pull/301#discussion_r1811575673 are more follow-ups than something to be done here.
The conflict is caused by: The user requested typing_extensions==4.2.0 mypy 1.11.2 depends on typing-extensions>=4.6.0 The user requested typing_extensions==4.2.0 mypy 1.11.1 depends on typing-extensions>=4.6.0 The user requested typing_extensions==4.2.0 mypy 1.11.0 depends on typing-extensions>=4.6.0
I unpinned typing_extensions
to just follow what mypy uses
@Avasam, thanks for your work here!
Work towards #296
Counterpart to #302 but this only touches configurations and infra. No stub change here except for
stubs/scipy-stubs/optimize/cython_optimize/__init__.pyi
(which is purely a mypy workaround otherwise it wouldn't even run due to same-name modules). So you can decide which you want first. There will be slight conflicts but I'll resolve them.typeCheckingMode
setting from IDE only to pyright configswarn-partial
setting from GitHub actions to pyright configsshow_column_numbers
in mypytests/__init__.py
stubs/scipy-stubs/optimize/cython_optimize/__init__.pyi
to get mypy to runtests/requirements.txt
tests/run_tests.sh
andtests/run_tests.cmd
into a Python script, simplified the commands, and update to run all the teststests/run_formatters.py
to run all code fixers and formatters