Skip to content

Conversation

@jscheffl
Copy link
Contributor

Following #57181 this is now a batch for all prek hooks that are "just" solely for providers to split out from root:

As prek is supporting monorepo now and go SDK was the front-runner, Airflow-Core is now the next piece that with this PR is proposed to be split-out from global pre-commit hook list into the provider space.

Note: This is the crown and msot probably atm the last thing to move. Careful review please!

Second attempt as a workarounf until the root cause of j178/prek#973 is clear. Added some notes for future follow-up

@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 Oct 26, 2025
@jscheffl jscheffl changed the title Feature/extract pre commit for airflow core 2 Feature/extract pre commit for airflow core (2nd attempt) Oct 26, 2025
@jscheffl
Copy link
Contributor Author

@potiuk Do you want to wait on #57310 or shall we merge 90% of the same in here? Then only the two compile-ui-assets would stay in the existing place as future cleanup.

@potiuk potiuk merged commit f69ba00 into apache:main Oct 26, 2025
63 checks passed
@github-actions
Copy link

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

Status Branch Result
v3-1-test Commit Link

You can attempt to backport this manually by running:

cherry_picker f69ba00 v3-1-test

This should apply the commit to the v3-1-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

@jscheffl
Copy link
Contributor Author

I think there is no need to backport, correct?

@potiuk
Copy link
Member

potiuk commented Oct 28, 2025

Sure :)

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.

2 participants