You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by ka...@apache.org on 2025/08/18 15:31:07 UTC

(airflow) 16/18: [v3-0-test] Fix prek teething errors for canary builds (#54583) (#54590)

This is an automated email from the ASF dual-hosted git repository.

kaxilnaik pushed a commit to branch v3-0-test
in repository https://gitbox.apache.org/repos/asf/airflow.git

commit 3538ed81b8d1b7c41a70e7a2c73e2a441fc02d98
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Sun Aug 17 16:10:23 2025 +0200

    [v3-0-test] Fix prek teething errors for canary builds (#54583) (#54590)
    
    After switching to prek in #54258 there are still few small things
    thet were only discovered in our canary builds:
    
    * installation of prek hooks does not necessarily create
      uv cache
    * autoupgrade is not yet implemented in prek
    (cherry picked from commit 703af5e5a1dc0f4911050e0d6317d5784be27426)
---
 .github/workflows/basic-tests.yml     | 14 ++++++++------
 .github/workflows/ci-image-checks.yml |  2 +-
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/basic-tests.yml b/.github/workflows/basic-tests.yml
index 32b7321cc60..65215396c05 100644
--- a/.github/workflows/basic-tests.yml
+++ b/.github/workflows/basic-tests.yml
@@ -257,12 +257,14 @@ jobs:
         with:
           python-version: ${{steps.breeze.outputs.host-python-version}}
           skip-prek-hooks: ${{ inputs.skip-prek-hooks }}
-      - name: "Autoupdate all prek hooks"
-        run: prek autoupdate --freeze
-      - name: "Autoupdate Lucas-C hooks to bleeding edge"
-        run: prek autoupdate --bleeding-edge --freeze --repo https://github.com/Lucas-C/pre-commit-hooks
-      - name: "Check if there are any changes in pre-commit hooks"
-        run: git diff --exit-code
+      # TODO(potiuk): enable this once prek supports automatic upgrade of hooks
+      #               after https://github.com/j178/prek/issues/35 is implemented§
+      # - name: "Autoupdate all prek hooks"
+      #   run: prek autoupdate --freeze
+      # - name: "Autoupdate Lucas-C hooks to bleeding edge"
+      #   run: prek autoupdate --bleeding-edge --freeze --repo https://github.com/Lucas-C/pre-commit-hooks
+      # - name: "Check if there are any changes in pre-commit hooks"
+      #   run: git diff --exit-code
       - name: "Run automated upgrade for black"
         run: >
           prek
diff --git a/.github/workflows/ci-image-checks.yml b/.github/workflows/ci-image-checks.yml
index 2d52aec1ad0..cd0c627ae11 100644
--- a/.github/workflows/ci-image-checks.yml
+++ b/.github/workflows/ci-image-checks.yml
@@ -151,7 +151,7 @@ jobs:
         if: inputs.canary-run == 'true'
       - name: "Prepare .tar file from prek cache"
         run: |
-          tar -C ~ -czf /tmp/cache-prek.tar.gz .cache/prek .cache/uv
+          tar -C ~ -czf /tmp/cache-prek.tar.gz .cache/prek
         shell: bash
         if: inputs.canary-run == 'true'
       - name: "Save prek cache"