Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jul 23, 2025

You can specify commit hash to apply when you build documentation. This allows to regenerate past version of the documentation by checking out the exact version tag that was used back then and applying the commit hash with fixes.

This might help in fixing issues like
#53646


^ 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 airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Jul 23, 2025
@potiuk
Copy link
Member Author

potiuk commented Jul 23, 2025

cc: @jedcunningham -> I think we also need to update the "Helm" releasing docs to use the breeze workflow-run publish-docs command. It becomes now quite powerful - including the option to patch past documentation with recent changes from main :D

@potiuk potiuk force-pushed the add-apply-commit-feature-to-docs-buildiung branch 6 times, most recently from cd5d693 to 2fb4b5b Compare July 23, 2025 15:21
You can specify commit hash to apply when you build documentation.
This allows to regenerate past version of the documentation by
checking out the exact version tag that was used back then and
applying the commit hash with fixes.

This might help in fixing issues like
#53646
@potiuk potiuk force-pushed the add-apply-commit-feature-to-docs-buildiung branch from 2fb4b5b to 217e02f Compare July 23, 2025 16:22
@potiuk
Copy link
Member Author

potiuk commented Jul 23, 2025

Ok. Few fixes - I also got some warnings that use-uv is not needed and indeed - action-breeze does not need use-uv any more so I removed it as well.

@potiuk
Copy link
Member Author

potiuk commented Jul 23, 2025

I will merge it when it gets green and test the full workflow with it.

@potiuk
Copy link
Member Author

potiuk commented Jul 23, 2025

I had to add some workaround for documentation released before we released airflow 3 (hive 9.0.0) because image stopped building for those - but I found a workaround how to fall-back and build it more manually

@potiuk potiuk merged commit d6a51e9 into main Jul 23, 2025
85 checks passed
@potiuk potiuk deleted the add-apply-commit-feature-to-docs-buildiung branch July 23, 2025 16:30
@github-actions
Copy link

Backport failed to create: v3-0-test. View the failure log Run details

Status Branch Result
v3-0-test Commit Link

You can attempt to backport this manually by running:

cherry_picker d6a51e9 v3-0-test

This should apply the commit to the v3-0-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

potiuk added a commit to potiuk/airflow that referenced this pull request Jul 23, 2025
…pache#53668)

You can specify commit hash to apply when you build documentation.
This allows to regenerate past version of the documentation by
checking out the exact version tag that was used back then and
applying the commit hash with fixes.

This might help in fixing issues like
apache#53646

(cherry picked from commit d6a51e9)
potiuk added a commit that referenced this pull request Jul 23, 2025
… publishing (#53668) (#53678)

You can specify commit hash to apply when you build documentation.
This allows to regenerate past version of the documentation by
checking out the exact version tag that was used back then and
applying the commit hash with fixes.

This might help in fixing issues like
#53646

(cherry picked from commit d6a51e9)
for APPLY_COMMIT in "${COMMIT_ARRAY[@]}"; do
echo "Applying commit ${APPLY_COMMIT}"
git fetch origin "${APPLY_COMMIT}"
git cherry-pick "${APPLY_COMMIT}"
Copy link
Member

Choose a reason for hiding this comment

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

does it cause any conflicts and fail when cherry-picking?

@gopidesupavan
Copy link
Member

thanks nice change :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants