Skip to content

Conversation

@jscheffl
Copy link
Contributor

@jscheffl jscheffl commented Dec 18, 2024

Reverts #44744

Seems to be the "quickest" way to fix main, will re-open a PR to merge back again with full tests after

FYI @vincbeck

Broken build: https://github.com/apache/airflow/actions/runs/12399970802/job/34616478712

@jscheffl
Copy link
Contributor Author

Note: I#ll have another PR as fix in 5min, maybe that saves a reversion...

@jscheffl
Copy link
Contributor Author

I propose to close this in favor of #45058

@potiuk
Copy link
Member

potiuk commented Dec 18, 2024

Yep.

@jscheffl
Copy link
Contributor Author

Other PR as quick fix failed, so I change my opinion and would propose the reversion route. Then we can re-apply and check fully.

@jscheffl jscheffl merged commit 2f92f82 into main Dec 18, 2024
40 checks passed
@potiuk potiuk deleted the revert-44744-vincbeck/compile_assets branch January 11, 2025 19:43
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
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.

3 participants