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

Add option to only check repos where mypy.ini is present #12

Closed
ostrokach opened this issue Nov 8, 2017 · 3 comments
Closed

Add option to only check repos where mypy.ini is present #12

ostrokach opened this issue Nov 8, 2017 · 3 comments

Comments

@ostrokach
Copy link

ostrokach commented Nov 8, 2017

I am playing around with MyPy with one of my Python packages, and so far linter-mypy has been working great!

However, all my other repositories do not have typehints and in fact cause MyPy to crash due to python/mypy#4111. Is there some way to disable linter-mypy for all repositories that do not have a mypy.ini file in the repository root (or through another way of detecting if a repo is using mypy?)

Thanks!

@elarivie
Copy link
Owner

@ostrokach
Hello Alexey,

Thanks for your feedback, I did reproduce the mypy internal issue that you obtain.

I notice that when a project is affected by a Mypy internal error a lot of annoying error pop-up are displayed. Which pretty much ruin the user experience and disabling linter-mypy is pretty much currently the only available option to keep working without those pop-up.

What I will implement:

  1. Improve the error pop-up so that any kind of Mypy internal error also display the full Mypy stacktrace to help bug reporting ("--show-traceback").

  2. Add a setting to turn-off Mypy internal error related pop-up (obviously when internal error occurs no lint warning will be shown for this specific Mypy run which will looks like as if they were deactivated for a given project).

  3. Add a button on the internal error pop-up to turn off pop-up of internal error.

@elarivie
Copy link
Owner

This is fix in linter-mypy v2.4.2

@ostrokach
Copy link
Author

The fix is working great! Thanks!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants