Skip to content

Enable qInfo tests for PySide6 #593

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

Merged
merged 1 commit into from
Mar 11, 2025
Merged

Conversation

penguinpee
Copy link
Contributor

PySide6 now exposes qInfo just like Qt6. This enables the qInfo
tests accordingly.

For reference see issue #232.

PySide6 now exposes `qInfo` just like Qt6. This enables the `qInfo`
tests accordingly.

For reference see issue pytest-dev#232.
@penguinpee
Copy link
Contributor Author

Some additional info: I noticed test_qinfo failing after PySide6 had been updated in Fedora a while ago. We are currently on release 6.8.2.1, which now exposes qInfo.

Copy link
Member

@nicoddemus nicoddemus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks! Appreciate it!

@nicoddemus nicoddemus merged commit ce1a689 into pytest-dev:master Mar 11, 2025
48 checks passed
The-Compiler added a commit that referenced this pull request Mar 25, 2025
This reverts commit ce1a689.

This change is incorrect as it breaks things with PySide < 6.8.2, but we support
running against older versions.

The next commit will fix this properly without breaking compatibility.

See #232
The-Compiler added a commit that referenced this pull request Mar 25, 2025
qInfo() has been added:
https://codereview.qt-project.org/c/pyside/pyside-setup/+/605100

Follow-up / alternative to #593 without breaking compatibility with older
PySide6 versions. See #232.
The-Compiler added a commit that referenced this pull request Mar 25, 2025
qInfo() has been added:
https://codereview.qt-project.org/c/pyside/pyside-setup/+/605100

Follow-up / alternative to #593 without breaking compatibility with older
PySide6 versions. See #232.
@The-Compiler
Copy link
Member

This breaks compatibility with PySide6 < 6.8.2. Reverted and pushed an alternative fix in #597.

The-Compiler added a commit that referenced this pull request Mar 25, 2025
This reverts commit ce1a689.

This change is incorrect as it breaks things with PySide < 6.8.2, but we support
running against older versions.

The next commit will fix this properly without breaking compatibility.

See #232
The-Compiler added a commit that referenced this pull request Mar 25, 2025
qInfo() has been added:
https://codereview.qt-project.org/c/pyside/pyside-setup/+/605100

Follow-up / alternative to #593 without breaking compatibility with older
PySide6 versions. See #232.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants