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

Update readme and copyright #42

Merged
merged 4 commits into from
Apr 12, 2016
Merged

Update readme and copyright #42

merged 4 commits into from
Apr 12, 2016

Conversation

t20100
Copy link
Member

@t20100 t20100 commented Apr 11, 2016

Closes #20
Closes #5

@vasole
Copy link
Member

vasole commented Apr 11, 2016

Shouldn't we say that the GUI depends on one of PyQt4, PySide or PyQt5 instead of mentioning one of them?

@t20100
Copy link
Member Author

t20100 commented Apr 12, 2016

It SHOULD work on PyQt5 and PySide, but so far it's only been truly tested on PyQt4.
To date, automatic tests with PySide do not work because a method is missing in QTest, and it's not really been tested on PyQt5.
I'd keep that for later.

@vasole
Copy link
Member

vasole commented Apr 12, 2016

Well, I have been using it with PyQt5.

By other hand, I do not consider an error to put in the readme where it should work. If it does not, it is a bug.

@vasole vasole closed this Apr 12, 2016
This was referenced Apr 12, 2016
@vasole vasole reopened this Apr 12, 2016
@vasole vasole merged commit e07b79e into silx-kit:master Apr 12, 2016
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.

None yet

2 participants