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

fix: upgrade PySide2 to be compatible with Python 3.10 #1226

Conversation

romainf-ubi
Copy link
Contributor

Link the Issue(s) this Pull Request is related to.
Fixes #1220

Summarize your change.
Right now, requirements_gui.txt requires PySide2 5.15.2 which is not compatible with Python 3.10.

This commit bumps it to PySide2 5.15.2.1, which is now compatible:

5.15.2:   Requires: Python >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, !=3.4.*, <3.10
5.15.2.1: Requires: Python >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, !=3.4.*, <3.11

Right now, requirements_gui.txt requires [PySide2 5.15.2](https://pypi.org/project/PySide2/5.15.2/)
which is not compatible with Python 3.10.

This commit bumps it to [PySide2 5.15.2.1](https://pypi.org/project/PySide2/5.15.2.1/),
which is now compatible:

```
5.15.2:   Requires: Python >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, !=3.4.*, <3.10
5.15.2.1: Requires: Python >=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, !=3.4.*, <3.11
```
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Nov 30, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link
Collaborator

@bcipriano bcipriano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just need to resolve the CLA check.

If this has been signed already, you might need to do a merge/rebase against master then re-push to the branch to get the check to run again. I don't remember if that's still needed or not.

@romainf-ubi
Copy link
Contributor Author

/easycla

@romainf-ubi
Copy link
Contributor Author

ok, all green for this one too!

@romainf-ubi
Copy link
Contributor Author

@bcipriano since this PR is now green and you approved it already, I guess you can merge it and close yours: #1235

@bcipriano bcipriano merged commit 1a920dc into AcademySoftwareFoundation:master Jan 12, 2023
@romainf-ubi romainf-ubi deleted the upstream-fix-python-3.10 branch January 12, 2023 18:10
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.

requirements_gui.txt is broken because of PySide2
2 participants