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

Prepwork pyside2 refactor #311

Merged
merged 3 commits into from
Nov 4, 2019
Merged

Prepwork pyside2 refactor #311

merged 3 commits into from
Nov 4, 2019

Conversation

dgdekoning
Copy link
Contributor

Prepare code for refactoring to pyside2 (#184).

These changes clear up PyQt specific naming and drops the use of QVariant.

No functional changes are made.

@dgdekoning dgdekoning added this to the Version 2.5.0 milestone Oct 31, 2019
@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 49.733% when pulling bcaae71 on dgdekoning:prep-pyside2-refactor into 60f4c55 on LCA-ActivityBrowser:master.

@dgdekoning dgdekoning changed the base branch from master to pyside2-refactor November 4, 2019 11:28
@dgdekoning dgdekoning merged commit a308cf4 into LCA-ActivityBrowser:pyside2-refactor Nov 4, 2019
bsteubing pushed a commit that referenced this pull request Nov 4, 2019
* Prepwork pyside2 refactor (#311)

* Refactor code to use PySide signal/slot nomenclature

* Use button reference instead of integer

* Remove QVariant references

* Replace PyQt5 with PySide2

Change 'flags' to 'f' for compatibility

* Change menu.exec to menu.exec_ for compatibility

* Specify setCheckState for compatibility

* Change recipe files to use pyside2

* Update automation pipeline scripts

* [skip ci] Clean-up imports

* [skip ci] Missed an exec to exec_ change

* [skip ci] Change licence from GPL to LGPL (#313)

* Replace GPL3 license file with LGPL3

* Update references to the license

* Add myself as main developer

* Update texts for CONTRIBUTING.md to reflect previous pytest changes

* Update README create/install commands to prefer conda-forge channel
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