Skip to content

Use more mypy_primer shards#15765

Open
Akuli wants to merge 1 commit into
mainfrom
primer6
Open

Use more mypy_primer shards#15765
Akuli wants to merge 1 commit into
mainfrom
primer6

Conversation

@Akuli
Copy link
Copy Markdown
Collaborator

@Akuli Akuli commented May 11, 2026

The CI on pull requests seems to take about 6 minutes, and the slowest thing is mypy_primer. Everything else is done in about 4 minutes. If this PR works like I intend, it will shorten the total CI time to about 4 minutes.

@github-actions
Copy link
Copy Markdown
Contributor

Diff from mypy_primer, showing the effect of this PR on open source code:

pandas (https://github.com/pandas-dev/pandas)
+ pandas/core/computation/ops.py:328: error: Need type annotation for "_binary_ops_dict" (hint: "_binary_ops_dict: dict[<type>, <type>] = ...")  [var-annotated]

@Akuli
Copy link
Copy Markdown
Collaborator Author

Akuli commented May 11, 2026

Timing seems to work fine, in this PR the last mypy_primer shard completed less than 30 seconds later than everything else. But why does this affect the output of mypy_primer?

@Akuli
Copy link
Copy Markdown
Collaborator Author

Akuli commented May 11, 2026

(I also just noticed that today I've been pushing directly here rather than my own fork. Oops :) )

@JelleZijlstra
Copy link
Copy Markdown
Member

This is a known flake where mypy is nondeterministic, Shantanu put up a fix in mypy.

@srittau
Copy link
Copy Markdown
Collaborator

srittau commented May 11, 2026

(I also just noticed that today I've been pushing directly here rather than my own fork. Oops :) )

No worries, I think a few maintainers prefer branches in this repo instead of their own copy, and that's fine. The main branch is protected against direct pushes anyway.

@Akuli Akuli marked this pull request as ready for review May 11, 2026 15:42
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.

3 participants