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: QWebChannel implementation and tests #103
Conversation
@@ -21,6 +22,29 @@ | |||
from spyder.utils.qthelpers import create_action, add_actions | |||
|
|||
from qtpy.QtWebEngineWidgets import WEBENGINE | |||
if WEBENGINE: | |||
from PyQt5.QtWebChannel import QWebChannel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we add support for QtWebChannel
in qtpy? I think it's better because PySide2 also supports Qt5.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was thinking exactly the same! I can send a PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, please do it.
@@ -28,7 +28,7 @@ dependencies: | |||
export PATH="$HOME/miniconda/bin:$PATH" && | |||
source activate test && | |||
conda install -q ciocheck -c spyder-ide --no-update-deps && | |||
if [ "$CIRCLE_NODE_INDEX" = "2" ]; then pip install -q spyder pyqt5 tornado pexpect; else conda install -q -y spyder && conda install tornado pexpect; fi && | |||
if [ "$CIRCLE_NODE_INDEX" = "2" ]; then pip install -q spyder pyqt5==5.8.0 tornado pexpect; else conda install -q -y spyder && conda install tornado pexpect; fi && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need to pin pyqt5
to 5.8.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried with 5.9.0, but it segfaults
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok
I'm in PyQt-5.9.0 / Windows , so I should not upgrade for now ? |
The problem was only related to our tests. |
Fixes #102