plotly / plotly.py

The interactive graphing library for Python :sparkles: This project now includes Plotly Express!
https://plotly.com/python/
MIT License
15.62k stars 2.51k forks source link

build+docs: use Python 3.11 and pin a recent version of pytest #4605

Closed gvwilson closed 1 month ago

gvwilson commented 1 month ago

Code PR

closes #4591

gvwilson commented 1 month ago

@emilykl not yet - I'd like to fix the broken tests on Python 3.11 first, then switch back to Python 3.12 and tackle the imp issue. Good news is that pyzmq 26.0.3 seems to install with Python 3.12.

gvwilson commented 1 month ago

Results on MacOS Sonoma 14.4.1 - I will update my PR tomorrow to (a) pin the pytest version and (b) modify contributing.md to say this has been tested on Python 8-11. Still have failing tests in several areas; I'll dig into those as well in separate PRs.

Python version existing pytest==3.5.1 updated pytest==8.1.1
3.8.19 fails: ImportError: cannot import name 'Config' from 'pytest' succeeds
3.9.19 fails: ImportError: cannot import name 'Config' from 'pytest' succeeds
3.10.14 fails: TypeError: required field "lineno" missing from alias succeeds
3.11.9 fails: TypeError: required field "lineno" missing from alias succeeds
gvwilson commented 1 month ago

PR updated: @LiamConnors can you please take a look at contributing.md and CHANGELOG.md and let me know?

gvwilson commented 1 month ago

@LiamConnors pushed a fix for the CHANGELOG (sorry about 8-11 instead of 3.8-3.11) - rebased to put it in the same commit as the other changes to hide my silly typo.