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

Mypy 730 compat #355

Merged
merged 2 commits into from
Nov 21, 2019
Merged

Mypy 730 compat #355

merged 2 commits into from
Nov 21, 2019

Conversation

rik
Copy link
Contributor

@rik rik commented Nov 18, 2019

My previous patch does not work because of this silly change I did after testing…

Somehow, I missed the --no-error-summary flag the first time I read #345. Thanks @ziyadedher for finding this!

Therefore, this PR reverts my previous attempt and passes one more flag.

I've tested this on mypy 0.720: it correctly ignores this flag. And it works with 0.740.

Mypy now adds a summary as the last line of the output. Pass a flag to not add this symmary.

Previous versions of mypy ignore this flag.

fix prospector-dev#345
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.8%) to 69.6% when pulling a22626e on rik:mypy-730-compat into 00a33d9 on PyCQA:develop.

@coveralls
Copy link

coveralls commented Nov 18, 2019

Coverage Status

Coverage decreased (-0.2%) to 73.233% when pulling 5804628 on rik:mypy-730-compat into 00a33d9 on PyCQA:develop.

@chocoelho
Copy link
Contributor

Thanks @rik. I should release a new dev version EOD with this.

@chocoelho chocoelho merged commit 0c4fe40 into prospector-dev:develop Nov 21, 2019
@rik rik deleted the mypy-730-compat branch February 10, 2024 16:30
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