Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jan 12, 2025

Since we are switching to Sphinx7, we need to build docs using Python 3.9. We need to make sure that Python 3.9 is default image.


^ 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.

@potiuk potiuk requested review from eladkal, jscheffl, kaxil and shahar1 and removed request for ashb and jedcunningham January 12, 2025 17:45
@potiuk
Copy link
Member Author

potiuk commented Jan 12, 2025

I think this should be as simple as that one line change @shahar1 :)

@potiuk potiuk force-pushed the build-docs-with-python-3.9 branch from 1ddf5ac to 05e45af Compare January 12, 2025 17:49
@potiuk
Copy link
Member Author

potiuk commented Jan 12, 2025

With some unit tests of course :)

Let's see - but after removal of the "pull_request_target" - such changes are quite a bit simpler as we have only "current" workflow - not two workflows to synchronize - one from PR and one from branch.

@potiuk potiuk force-pushed the build-docs-with-python-3.9 branch 2 times, most recently from f3c42e1 to dcb299f Compare January 12, 2025 18:13
@potiuk
Copy link
Member Author

potiuk commented Jan 12, 2025

OK. One more place where 3.8 had to be changed to 3.9 - we have static checks using it. Also some docs - just in case and error messages pointing to build 3.9 shoudl be updated. Will make @jscheffl 3.8-3.9 PR smaller :)

@potiuk potiuk force-pushed the build-docs-with-python-3.9 branch from dcb299f to ebf4787 Compare January 12, 2025 19:26
@potiuk potiuk added the legacy api Whether legacy API changes should be allowed in PR label Jan 12, 2025
@potiuk potiuk closed this Jan 12, 2025
@potiuk potiuk reopened this Jan 12, 2025
Since we are switching to Sphinx7, we need to build docs using
Python 3.9. We need to make sure that Python 3.9 is default image.
@potiuk potiuk force-pushed the build-docs-with-python-3.9 branch from ebf4787 to 8eeb012 Compare January 12, 2025 19:51
@potiuk potiuk requested a review from o-nikolas as a code owner January 12, 2025 19:51
@potiuk potiuk merged commit 0ecc0cd into apache:v2-10-test Jan 12, 2025
59 checks passed
@potiuk potiuk deleted the build-docs-with-python-3.9 branch January 12, 2025 20:45
@utkarsharma2 utkarsharma2 added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Jan 28, 2025
@utkarsharma2 utkarsharma2 added this to the Airflow 2.10.5 milestone Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) legacy api Whether legacy API changes should be allowed in PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants