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.

Sign up for GitHub

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

Jump to bottom

typing.Callable #5087

Closed
kosyachniy opened this issue Sep 28, 2021 · 0 comments · Fixed by #5089
Closed

typing.Callable #5087

kosyachniy opened this issue Sep 28, 2021 · 0 comments · Fixed by #5089
Assignees
@cdce8p
Labels
Bug 🪲 False Positive 🦟 A message is emitted but nothing is wrong with the code Needs astroid Brain 🧠 Needs a brain tip in astroid (then an astroid upgrade)
Milestone
2.12.0

Comments

@kosyachniy
Copy link

Bug description

# Steps to reproduce
from typing import Callable

# Current behavior
print(isinstance([], Callable))

I get error:
`[isinstance-second-argument-not-valid-type] Second argument of isinstance is not a type`

Configuration

-

Command used

isinstance

Pylint output

[isinstance-second-argument-not-valid-type] Second argument of isinstance is not a type

Expected behavior

usual

Pylint version

pylint 2.11.1
astroid 2.8.0
Python 3.9.2 (default, Feb 24 2021, 05:06:40) 
[Clang 12.0.0 (clang-1200.0.32.29)]

OS / Environment

MacOS

Additional dependencies

No response
@kosyachniy kosyachniy added Bug 🪲 Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Sep 28, 2021
@cdce8p cdce8p added False Positive 🦟 A message is emitted but nothing is wrong with the code Needs astroid Brain 🧠 Needs a brain tip in astroid (then an astroid upgrade) and removed Needs astroid Brain 🧠 Needs a brain tip in astroid (then an astroid upgrade) Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Sep 28, 2021
@cdce8p cdce8p added this to the 2.11.2 milestone Sep 28, 2021
@cdce8p cdce8p mentioned this issue Sep 28, 2021
Merged
@cdce8p cdce8p self-assigned this Sep 28, 2021
@cdce8p cdce8p mentioned this issue Sep 28, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cdce8p cdce8p

Labels
Bug 🪲 False Positive 🦟 A message is emitted but nothing is wrong with the code Needs astroid Brain 🧠 Needs a brain tip in astroid (then an astroid upgrade)
Projects
None yet
Milestone
2.12.0
Development

Successfully merging a pull request may close this issue.

Add test for typing alias in isinstance calls cdce8p/pylint
2 participants
@kosyachniy @cdce8p