Skip to content

Conversation

@The-Compiler
Copy link
Member

Since PyQt5 5.6 it's available as binary wheel including Qt on PyPI.
This will make it easier for contributors too, as they can simply run
tox.

Since PyQt5 5.6 it's available as binary wheel including Qt on PyPI.
This will make it easier for contributors too, as they can simply run
tox.
@coveralls
Copy link

coveralls commented Oct 26, 2016

Coverage Status

Coverage decreased (-2.6%) to 97.366% when pulling 67b8cad on The-Compiler:pypi into af46be9 on pytest-dev:master.

@nicoddemus
Copy link
Member

Thanks!

Apparently the pyqt5 binaries are not available for py34. If that's the case, I think it is OK to drop testing py34 with pyqt5.

@The-Compiler
Copy link
Member Author

Alternatively I could use the distribution's PyQt5 for Python 3.4 - what would you prefer?

@nicoddemus
Copy link
Member

I'm OK with dropping py34-pyqt5 to simplify things. py34 and py35 are pretty much compatible, plus we are testing py34 with pyside.

To recap:

Windows

  • py34-pyqt5 (FAILED)
  • py35-pyqt5 (OK)
  • py34-pyqt4 (OK)
  • py27-pyqt4 (OK)
  • py27-pyside (OK)
  • py34-pyside (OK)

Linux

  • py27-pyqt4 (OK)
  • py27-pyside (OK)
  • py34-pyside (OK)
  • py27-pyside (OK)
  • py34-pyqt5 (FAILED)

So I would be OK with dropping py34-pyqt5 in both platforms and add py35-pyqt5 to Travis instead. What do you think?

@The-Compiler
Copy link
Member Author

Sounds good, I hope I adjusted everything correctly! I also adjusted the text in the README.

@coveralls
Copy link

coveralls commented Oct 30, 2016

Coverage Status

Coverage decreased (-2.5%) to 97.451% when pulling 7e45269 on The-Compiler:pypi into af46be9 on pytest-dev:master.

@coveralls
Copy link

coveralls commented Oct 30, 2016

Coverage Status

Coverage decreased (-2.5%) to 97.451% when pulling 7e45269 on The-Compiler:pypi into af46be9 on pytest-dev:master.

@nicoddemus
Copy link
Member

No longer relevant now that we test with conda 😁

@nicoddemus nicoddemus closed this Sep 15, 2017
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