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 .readthedocs.yml to v2 #267

Merged
merged 10 commits into from
Sep 1, 2023
Merged

update .readthedocs.yml to v2 #267

merged 10 commits into from
Sep 1, 2023

Conversation

thijstriemstra
Copy link
Collaborator

No description provided.

rm-hull
rm-hull previously approved these changes Sep 1, 2023
rm-hull
rm-hull previously approved these changes Sep 1, 2023
@thijstriemstra
Copy link
Collaborator Author

thijstriemstra commented Sep 1, 2023

Looks like the doc build is failing on Python 3.7 because it's using an old version of Sphinx (because python 3.7 support was dropped in sphinx v6.0 so no way to upgrade it either. The actual string it's failing on is not coming from luma.core but the intersphinx for Pillow, which likely was fixed in Sphinx at some point.

Solution is skipping doc build on Python 3.7 (which we should drop support for sometime next year IMO).

@@ -39,6 +40,7 @@ jobs:
python_env=$(echo $TOX_ENV | sed -e s/-dev$//)
python -m tox -e ${python_env}
- name: QA
if: ${{ steps.setuppy.outputs.python-version > 7 }}
Copy link
Owner

Choose a reason for hiding this comment

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

🤫

@rm-hull rm-hull merged commit caeaae5 into master Sep 1, 2023
8 checks passed
@rm-hull rm-hull deleted the rtd-v2 branch September 1, 2023 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants