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

PR: Add QtQuickWidgets #179

Merged
merged 2 commits into from
Jan 11, 2019
Merged

Conversation

KurtJacobson
Copy link
Contributor

This PR adds support for QtQuickWidgets.

@ccordoba12
Copy link
Member

Please skip

qtpy/tests/test_patch_qcombobox.py::test_model_item

in Pyside2 to see if our tests pass there.

@ccordoba12
Copy link
Member

@KurtJacobson, please remove your last commit. I'm going to fix the failures with our tests in PR #180.

@KurtJacobson
Copy link
Contributor Author

@ccordoba12 Excellent! I will revert 104b33c.

@ccordoba12 ccordoba12 added this to the v1.6 milestone Jan 10, 2019
@ccordoba12
Copy link
Member

@KurtJacobson, please rebase or merge with latest master to get the fixes.

@ccordoba12
Copy link
Member

Also, a test is missing for your change.

@KurtJacobson
Copy link
Contributor Author

@ccordoba12 Will do, and I'll add the tests. Thank you!

@ccordoba12 ccordoba12 changed the title Add QtQuickWidgets PR; Add QtQuickWidgets Jan 11, 2019
@ccordoba12 ccordoba12 changed the title PR; Add QtQuickWidgets PR: Add QtQuickWidgets Jan 11, 2019
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

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

Thanks @KurtJacobson!

@ccordoba12 ccordoba12 merged commit 6cc245a into spyder-ide:master Jan 11, 2019
@ccordoba12
Copy link
Member

1.6.0 is out, with this change. Enjoy!

@KurtJacobson
Copy link
Contributor Author

@ccordoba12 Thanks for the very quick release!

@KurtJacobson KurtJacobson deleted the AddQtQuickWidgets branch January 12, 2019 01:40
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.

2 participants