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

linter doesn't dig context managers #1280

Closed
dimaqq opened this issue Feb 7, 2019 · 2 comments
Closed

linter doesn't dig context managers #1280

dimaqq opened this issue Feb 7, 2019 · 2 comments

Comments

@dimaqq
Copy link

dimaqq commented Feb 7, 2019

test.py:

with open("/dfga", "r") as fin:
    fin.read()

invocation:

> python -m jedi linter test.py
/path/to/test.py:1:27: E2 NameError: name 'fin' is not defined.
@davidhalter
Copy link
Owner

Thanks for trying the linter. I'll keep it open. However note that the linter is currently not being developed and I doubt that it will be for another year. We probably will create a database index #1059 first and then do proper linting. In the current iteration the linter is just not good enough anyway, so I don't even bother.

@davidhalter
Copy link
Owner

Looks like this was a more general error.

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