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

False positive E1111 (assignment-from-no-return) for sys.stdin.read() #2571

Closed
jeffwidman opened this issue Oct 25, 2018 · 2 comments
Closed
Labels
Milestone

Comments

@jeffwidman
Copy link

jeffwidman commented Oct 25, 2018

False positive E1111 (assignment-from-no-return) for sys.stdin.read().

Code:

if __name__ == "__main__":
    import sys
    data = sys.stdin.read()

The final line here throws E1111:

E: Assigning to function call which doesn't return (assignment-from-no-return)

However, sys.stdin.read() does return, so this is a false positive.

Pylint version: 2.1.1

Weirdly, this is only thrown on python 3.5 and 3.6, not on 2.7 or 3.4.

You can see the full test matrix here: https://travis-ci.org/dpkp/kafka-python/builds/446012872

@PCManticore
Copy link
Contributor

Thanks, this is definitely a bug!

jeffwidman added a commit to dpkp/kafka-python that referenced this issue Oct 26, 2018
Temporarily workaround pylint-dev/pylint#2571 so that we can stop pinning `pylint` (#1611)
jeffwidman added a commit to dpkp/kafka-python that referenced this issue Oct 26, 2018
Temporarily workaround pylint-dev/pylint#2571 so that we can stop pinning `pylint` (#1611)
jeffwidman added a commit to dpkp/kafka-python that referenced this issue Oct 26, 2018
Temporarily workaround pylint-dev/pylint#2571 so that we can stop pinning `pylint`.
jeffwidman added a commit to dpkp/kafka-python that referenced this issue Oct 27, 2018
Temporarily workaround pylint-dev/pylint#2571 so that we can stop pinning `pylint`.
@DanielNoord
Copy link
Collaborator

@Pierre-Sassoulas Can't reproduce this anymore!

@Pierre-Sassoulas Pierre-Sassoulas added this to the 2.10.0 milestone Aug 5, 2021
jeffwidman added a commit to dpkp/kafka-python that referenced this issue Aug 5, 2021
jeffwidman added a commit to dpkp/kafka-python that referenced this issue Aug 5, 2021
DoubleRampage added a commit to DoubleRampage/MBA-kafka that referenced this issue Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants