Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix function call message change for small number of args - #21432

#21432
Merged
hauntsaninja merged 2 commits into
masterpython/mypy:masterfrom
issue-21427python/mypy:issue-21427Copy head branch name to clipboard
May 10, 2026
Merged

Fix function call message change for small number of args#21432
hauntsaninja merged 2 commits into
masterpython/mypy:masterfrom
issue-21427python/mypy:issue-21427Copy head branch name to clipboard

Conversation

@sobolevn

@sobolevn sobolevn commented May 7, 2026

Copy link
Copy Markdown
Member

Refs #21427

I used 2 as a magic number, because in my opinion it is easier to tell the difference when all errors show up for the small number of args.

Any feedback is welcome 👍

@sobolevn
sobolevn requested review from JukkaL and KevinRK29 May 7, 2026 13:35
@github-actions

This comment has been minimized.

@KevinRK29

Copy link
Copy Markdown
Collaborator

makes sense to me, thanks!

@github-actions

Copy link
Copy Markdown
Contributor

According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅

@hauntsaninja
hauntsaninja merged commit 38b1bdf into master May 10, 2026
25 checks passed
@hauntsaninja
hauntsaninja deleted the issue-21427 branch May 10, 2026 06:50
@JukkaL JukkaL mentioned this pull request May 10, 2026
2 tasks
JukkaL pushed a commit that referenced this pull request May 10, 2026
Refs #21427

I used `2` as a magic number, because in my opinion it is easier to tell
the difference when all errors show up for the small number of args.

---------

Co-authored-by: hauntsaninja <hauntsaninja@gmail.com>
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.