Skip to content

libregrtest: Improve type hints; explore running mypy in CI #109413

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
AlexWaygood opened this issue Sep 14, 2023 · 5 comments
Open

libregrtest: Improve type hints; explore running mypy in CI #109413

AlexWaygood opened this issue Sep 14, 2023 · 5 comments
Labels
tests Tests in the Lib/test dir type-feature A feature request or enhancement

Comments

@AlexWaygood
Copy link
Member

AlexWaygood commented Sep 14, 2023

Feature or enhancement

Proposal:

The code in Lib/test/libregrtest already has a smattering of type hints in it, but doesn't currently pass type checking. @vstinner, the primary maintainer of libregrtest, expressed interest to me in seeing whether we could run mypy on libregrtest in CI.

This issue is for tracking progress towards that.

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Linked PRs

@AlexWaygood AlexWaygood added type-feature A feature request or enhancement tests Tests in the Lib/test dir labels Sep 14, 2023
@AlexWaygood
Copy link
Member Author

We currently do not have any projects inside the Lib/ directory that are mypy-checked in CI: currently, all the mypy-checked projects we have in CPython live in the Tools/. If we want to actually run mypy on libregrtest in CI, I think it should be discussed on Discourse first, to check if anybody has any objections to this idea.

@AlexWaygood AlexWaygood changed the title libregrtest: Improve type hints; explore running mypy on CI libregrtest: Improve type hints; explore running mypy in CI Sep 14, 2023
AlexWaygood added a commit to AlexWaygood/cpython that referenced this issue Sep 15, 2023
hauntsaninja pushed a commit that referenced this issue Sep 18, 2023
Improve the mypy config file for libregrtest
csm10495 pushed a commit to csm10495/cpython that referenced this issue Sep 28, 2023
AlexWaygood added a commit that referenced this issue Nov 30, 2023
AlexWaygood added a commit to AlexWaygood/cpython that referenced this issue Dec 1, 2023
vstinner added a commit to vstinner/cpython that referenced this issue Dec 1, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Dec 1, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Dec 1, 2023
AlexWaygood pushed a commit that referenced this issue Dec 1, 2023
…2594)

gh-109413: regrtest: add WorkerRunTests class (GH-112588)
(cherry picked from commit f8ff80f)

Co-authored-by: Victor Stinner <[email protected]>
AlexWaygood pushed a commit that referenced this issue Dec 1, 2023
…2593)

gh-109413: regrtest: add WorkerRunTests class (GH-112588)
(cherry picked from commit f8ff80f)

Co-authored-by: Victor Stinner <[email protected]>
AlexWaygood added a commit that referenced this issue Dec 1, 2023
AlexWaygood added a commit to AlexWaygood/cpython that referenced this issue Dec 1, 2023
sobolevn added a commit that referenced this issue Nov 14, 2024
sobolevn added a commit to sobolevn/cpython that referenced this issue Nov 15, 2024
sobolevn added a commit that referenced this issue Nov 18, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Nov 18, 2024
…_workers` (pythonGH-126855)

(cherry picked from commit a1d9c8a)

Co-authored-by: sobolevn <[email protected]>
Co-authored-by: Alex Waygood <[email protected]>
sobolevn added a commit that referenced this issue Nov 18, 2024
…n_workers` (GH-126855) (#126967)

gh-109413: Enable `strict_optional = true` for `libregrtest/run_workers` (GH-126855)
(cherry picked from commit a1d9c8a)

Co-authored-by: sobolevn <[email protected]>
Co-authored-by: Alex Waygood <[email protected]>
sobolevn added a commit to sobolevn/cpython that referenced this issue Nov 19, 2024
sobolevn added a commit to sobolevn/cpython that referenced this issue Nov 19, 2024
sobolevn added a commit to sobolevn/cpython that referenced this issue Nov 19, 2024
sobolevn added a commit that referenced this issue Nov 19, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Nov 19, 2024
…hecking `libregrtest` (pythonGH-127033)

(cherry picked from commit 824afbf)

Co-authored-by: sobolevn <[email protected]>
vstinner added a commit to vstinner/cpython that referenced this issue Nov 22, 2024
Skip threads which are not running.
vstinner added a commit to vstinner/cpython that referenced this issue Nov 22, 2024
Skip threads which are not running.
vstinner added a commit that referenced this issue Nov 22, 2024
Skip threads which are not running.
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Nov 22, 2024
Skip threads which are not running.
(cherry picked from commit 0cb2017)

Co-authored-by: Victor Stinner <[email protected]>
vstinner added a commit that referenced this issue Nov 22, 2024
gh-109413: Fix libregrtest get_running() (GH-127153)

Skip threads which are not running.
(cherry picked from commit 0cb2017)

Co-authored-by: Victor Stinner <[email protected]>
sobolevn added a commit that referenced this issue Nov 24, 2024
…checking `libregrtest` (GH-127033) (#127034)

gh-109413: Enable mypy's `disallow_any_generics` setting when checking `libregrtest` (GH-127033)
(cherry picked from commit 824afbf)

Co-authored-by: sobolevn <[email protected]>
picnixz pushed a commit to picnixz/cpython that referenced this issue Dec 8, 2024
picnixz pushed a commit to picnixz/cpython that referenced this issue Dec 8, 2024
sobolevn added a commit that referenced this issue Jan 8, 2025
…126388)

gh-109413: Add more type hints to `libregrtest` (GH-126352)
(cherry picked from commit bfc1d25)

Co-authored-by: sobolevn <[email protected]>
ebonnal pushed a commit to ebonnal/cpython that referenced this issue Jan 12, 2025
ebonnal pushed a commit to ebonnal/cpython that referenced this issue Jan 12, 2025
ebonnal pushed a commit to ebonnal/cpython that referenced this issue Jan 12, 2025
ebonnal pushed a commit to ebonnal/cpython that referenced this issue Jan 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Tests in the Lib/test dir type-feature A feature request or enhancement
Projects
None yet
Development

No branches or pull requests

2 participants