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 to Ubuntu 24.04 for readthedocs #16891

Merged
merged 2 commits into from
Oct 16, 2024
Merged

Conversation

radarhere
Copy link
Contributor

# Read the Docs configuration file
# See https://docs.readthedocs.io/en/stable/config-file/v2.html for details
# Required
version: 2
build:
os: ubuntu-22.04

https://docs.readthedocs.io/en/stable/config-file/v2.html mentions that 'ubuntu-24.04' can now be used. However, perhaps better than that is 'ubuntu-lts-latest', to automatically stay on the latest available OS.

@radarhere radarhere requested a review from a team as a code owner October 15, 2024 12:10
Copy link
Member

@miketheman miketheman left a comment

Choose a reason for hiding this comment

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

Thanks for the suggestion.

We'd prefer to have the underlying version "known" and using the lts version means that it could swap out underneath us.

@radarhere radarhere changed the title Use latest Ubuntu for readthedocs Update to Ubuntu 24.04 for readthedocs Oct 16, 2024
@radarhere
Copy link
Contributor Author

Ok, sure. I've changed the commit to use 'ubuntu-24.04' instead.

Copy link
Member

@miketheman miketheman left a comment

Choose a reason for hiding this comment

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

Thanks!

@miketheman miketheman enabled auto-merge (squash) October 16, 2024 20:47
@miketheman miketheman merged commit 694286e into pypi:main Oct 16, 2024
18 checks passed
@radarhere radarhere deleted the patch-1 branch October 16, 2024 20:56
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.

2 participants