Skip to content

Enable mypy parallel workers in pre-commit.#2992

Merged
adamtheturtle merged 3 commits intomainfrom
enable-mypy-parallel-workers-pre-commit
May 7, 2026
Merged

Enable mypy parallel workers in pre-commit.#2992
adamtheturtle merged 3 commits intomainfrom
enable-mypy-parallel-workers-pre-commit

Conversation

@adamtheturtle
Copy link
Copy Markdown
Member

@adamtheturtle adamtheturtle commented May 7, 2026

Summary

  • run the mypy pre-push hook with --num-workers=4 to enable parallel workers
  • run mypy-docs via doccmd with a command string that also includes --num-workers=4

Test plan

  • pre-commit hooks run successfully on commit (including yamlfix)
  • uv run --extra=dev -m mypy --num-workers=4 (executes; currently hits existing mypy internal error in repo)

Made with Cursor


Note

Low Risk
Low risk config-only change that only affects local pre-push type-checking execution; main impact is potential nondeterminism/perf differences in mypy output.

Overview
Updates the pre-push pre-commit hooks to run mypy with --num-workers=4, enabling parallel type-checking.

Also adjusts the mypy-docs doccmd invocation so the embedded mypy command includes the same --num-workers=4 flag.

Reviewed by Cursor Bugbot for commit 2d87cce. Bugbot is set up for automated code reviews on this repo. Configure here.

Co-authored-by: Cursor <cursoragent@cursor.com>
Keep `mypy-docs` aligned with `mypy` by keeping `--num-workers=4` inline in the mypy command argument.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@adamtheturtle adamtheturtle deployed to development May 7, 2026 12:36 — with GitHub Actions Active
@adamtheturtle adamtheturtle merged commit 0a25fd6 into main May 7, 2026
16 checks passed
@adamtheturtle adamtheturtle deleted the enable-mypy-parallel-workers-pre-commit branch May 7, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant