-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
Upgrade Sphinx to 7.3.7 #39449
Upgrade Sphinx to 7.3.7 #39449
Conversation
closes apache#31963 All the linked issues on the above have been resolved
One wrinkle: Lines 160 to 163 in 9a0d9c5
|
Other related issue/PR: |
FYI @kaxil . From what I remember (see my old PR) the biggest problem there was to make appropriate changes to the theme so that the document "look" right. The change in docutils needed means that the generated documentation has a different HTML structure (different tags are used to construct the headers etc. ) and we need to update/rebuild CSS and release a new theme to make it looks good (and do it in the way to not break past released documentation as the theme is shared). |
Yeah, I looked at the PR in apache/airflow-site#843, planning to circle back and check with @utkarsharma2 too on the CSS changes and stuff. Not immediately but hopefully we can fix this soon'ish |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions. |
closes #31963
All the linked issues on the above have been resolved
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rst
or{issue_number}.significant.rst
, in newsfragments.