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.

Sign up for GitHub

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

Jump to bottom

7.4.x pluggy CI builds fail #11293

Closed
The-Compiler opened this issue Aug 7, 2023 · 2 comments
Closed

7.4.x pluggy CI builds fail #11293

The-Compiler opened this issue Aug 7, 2023 · 2 comments
Labels
type: bug problem that needs to be addressed type: infrastructure improvement to development/releases/CI structure

Comments

@The-Compiler
Copy link
Member

see e.g. https://github.com/pytest-dev/pytest/actions/runs/5772741776/job/15648272495?pr=11286

INFO: pip is looking at multiple versions of pluggy to determine which version is compatible with other requirements. This could take a while.
py37-pluggymain-pylib-xdist: exit 1 (11.93 seconds) /home/runner/work/pytest/pytest> python -I -m pip install 'pluggy@ git+https://github.com/pytest-dev/pluggy.git' 'py>=1.8.2' 'pytest-xdist>=2.1.0' -e . pid=1883
  Running command git clone --filter=blob:none --quiet https://github.com/pytest-dev/pluggy.git /tmp/pip-install-i9cw8ots/pluggy_51a0b7decb294cd0865e1b815f453f8b
ERROR: Package 'pluggy' requires a different Python: 3.7.17 not in '>=3.8'
  py37-pluggymain-pylib-xdist: FAIL code 1 (12.27 seconds)

Not sure what to do there... should we just upgrade those pluggy builds from py37 to py38 for CI?
@The-Compiler The-Compiler added type: bug problem that needs to be addressed type: infrastructure improvement to development/releases/CI structure labels Aug 7, 2023
This was referenced Aug 7, 2023
Merged
Merged
@bluetech
Copy link
Member

bluetech commented Aug 7, 2023

should we just upgrade those pluggy builds from py37 to py38 for CI?

IMO, yes, should just change py37 -> py38, since the idea is to test against the latest pluggy.

The-Compiler added a commit to The-Compiler/pytest that referenced this issue Aug 7, 2023
@The-Compiler
ci: Use Python 3.8 to test latest pluggy
ba40975 
Pluggy dropped Python 3.7 support.
Also see 165fbbd

Fixes pytest-dev#11293
@The-Compiler The-Compiler mentioned this issue Aug 7, 2023
Merged
@Pierre-Sassoulas
Copy link
Member

Closing as fixed in #11294

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug problem that needs to be addressed type: infrastructure improvement to development/releases/CI structure
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@The-Compiler @bluetech @Pierre-Sassoulas