-
Notifications
You must be signed in to change notification settings - Fork 212
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
855 pyside2 #898
855 pyside2 #898
Conversation
We just copied all the requirements from the file requirements.txt so that docs has what it may need for CI testing in a virtual env. |
It is not elegant. I know. However, it allows us to use the next version of tox. |
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.
LGTM, we need to fix qiskit-metal's init.py so that we can set skip_install = true in tox.ini for the docs environment, but this should suffice for now. PLEASE NOTE: Setting skip_install = false makes sure we are installing dependencies into the tox environment with setup.py. Because setup.py installs dependencies from requirements.txt (not requirements-dev.txt), all of qiskit-metal's dependencies enter the tox environment in this way when skip_install = false.
What are the issues this pull addresses (issue numbers / links)?
Did you add tests to cover your changes (yes/no)?
Did you update the documentation accordingly (yes/no)?
Did you read the CONTRIBUTING document (yes/no)?
Summary
Details and comments