Skip to content
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

Set soft error limit default to -1 (unlimited) #15138

Merged

Conversation

rohitsanj
Copy link
Contributor

@rohitsanj rohitsanj commented Apr 25, 2023

Fixes #14915

When there are more than 200 errors to report, mypy "softly" trims the output without notifying the user that it did so. We want to currently disable this by setting the --soft-error-limit default to -1 which would list out all the errors.

@rohitsanj rohitsanj changed the title Set MANY_ERRORS_THRESHOLD default to -1 (unlimited) Set soft error limit default to -1 (unlimited) Apr 25, 2023
Copy link
Collaborator

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing!

@github-actions

This comment has been minimized.

mypy/defaults.py Outdated Show resolved Hide resolved
@rohitsanj rohitsanj requested a review from ikonst April 25, 2023 23:37
@github-actions
Copy link
Contributor

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

@hauntsaninja hauntsaninja merged commit bd2a641 into python:master Apr 26, 2023
20 checks passed
hauntsaninja pushed a commit that referenced this pull request Nov 23, 2023
Default value of `MANY_ERRORS_THRESHOLD` was set to `-1` in
#15138, which is also the default
value of the `--soft-error-limit` CLI option. However the CLI docs were
not updated accordingly.
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.

Default "--soft-error-limit" behavior causes tons of headache for mypy newbies with large code bases
3 participants