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

no-member with Qt signals using PySide6 #5378

Closed
EricEzaM opened this issue Nov 23, 2021 · 4 comments · Fixed by pylint-dev/astroid#1654
Closed

no-member with Qt signals using PySide6 #5378

EricEzaM opened this issue Nov 23, 2021 · 4 comments · Fixed by pylint-dev/astroid#1654
Labels
Bug 🪲 Duplicate 🐫 Duplicate of an already existing issue False Positive 🦟 A message is emitted but nothing is wrong with the code

Comments

@EricEzaM
Copy link

EricEzaM commented Nov 23, 2021

Bug description

Exactly the same as #2585.

Configuration

Create signal_test.py containing:

from PySide6 import QtWidgets
app = QtWidgets.QApplication()
app.focusChanged.connect(lambda x: None)

Command used

pylint .\signal_test.py --extension-pkg-whitelist=PySide6

Pylint output

test.py:4:0: E1101: Method 'focusChanged' has no 'connect' member (no-member)

Expected behavior

No Errors

Pylint version

pylint 2.11.1
astroid 2.8.4
Python 3.8.10

OS / Environment

Win 10

As mentioned, this was addressed in #2585 for PySide2

Additional dependencies

PySide6 6.2.0

@EricEzaM EricEzaM added Bug 🪲 Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Nov 23, 2021
@Pierre-Sassoulas Pierre-Sassoulas added False Positive 🦟 A message is emitted but nothing is wrong with the code and removed Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Jan 25, 2022
@BoniLindsley
Copy link

I think the check given in pylint-dev/astroid#900 (comment) would fix this. Specifically line 4 of the code snippet.

is_pyside_node = node.qname().partition(".")[0] in ("PySide2", "PySide6")

That pull request might need to be adopted with this in mind, or needs to be merged first, for this to be added, I think.

@adam-grant-hendry
Copy link
Contributor

I am encountering the same issue with QAbstractAnimation.finished signals.

@adam-grant-hendry
Copy link
Contributor

adam-grant-hendry commented Jun 22, 2022

I think the check given in PyCQA/astroid#900 (comment) would fix this. Specifically line 4 of the code snippet.

is_pyside_node = node.qname().partition(".")[0] in ("PySide2", "PySide6")

That pull request might need to be adopted with this in mind, or needs to be merged first, for this to be added, I think.

I can confirm that check fixed the problem on my end.

+1 to have the fix merged in PR 900.

@jacobtylerwalls
Copy link
Member

Duplicate of #4040

(closing now as it's difficult to link two pylint issues to an astroid PR for auto-closing later)

@jacobtylerwalls jacobtylerwalls closed this as not planned Won't fix, can't repro, duplicate, stale Jun 24, 2022
@jacobtylerwalls jacobtylerwalls added the Duplicate 🐫 Duplicate of an already existing issue label Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🪲 Duplicate 🐫 Duplicate of an already existing issue False Positive 🦟 A message is emitted but nothing is wrong with the code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants