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

Use python 3.11 on RTD #2231

Merged
merged 2 commits into from
Mar 18, 2024
Merged

Use python 3.11 on RTD #2231

merged 2 commits into from
Mar 18, 2024

Conversation

natestemen
Copy link
Member

@natestemen natestemen commented Mar 14, 2024

Description

Read The Docs is using python 3.10 to build our documentation. This was originally part of our upgrade to support python 3.11 (#1663), but we went forward without it because there was some troubles with python 3.11 on RTD.

This is only one data point, but latest on RTD is building in 2697 seconds, and this PR is building in 2351 seconds. 5 minute improvement! Thanks python 3.11! Now if we can get our RTD build below 30 min that would be really cool, but something to think about down the line.

fixes #1790

@natestemen natestemen added documentation Improvements or additions to documentation tech debt Technical Debt that should be paid off! labels Mar 14, 2024
@natestemen natestemen self-assigned this Mar 14, 2024
Copy link

codecov bot commented Mar 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.28%. Comparing base (b9036cf) to head (e1c0cf9).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2231   +/-   ##
=======================================
  Coverage   98.28%   98.28%           
=======================================
  Files          90       90           
  Lines        4257     4257           
=======================================
  Hits         4184     4184           
  Misses         73       73           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@natestemen natestemen requested a review from cosenal March 18, 2024 16:44
@natestemen natestemen merged commit e5bc4ea into master Mar 18, 2024
18 checks passed
@natestemen natestemen deleted the nts-rtd branch March 18, 2024 16:56
@cosenal
Copy link
Contributor

cosenal commented Mar 18, 2024

Just to add another data point from a local build on my machine
make docs-clean 962.92s user 424.57s system 193% cpu 11:57.48 total

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation tech debt Technical Debt that should be paid off!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use python 3.11 for RTD builds
2 participants