j178 commented on code in PR #54258:
URL: https://github.com/apache/airflow/pull/54258#discussion_r2277856068


##########
.github/actions/install-prek/action.yml:
##########
@@ -16,38 +16,33 @@
 # under the License.
 #
 ---
-name: 'Install pre-commit'
-description: 'Installs pre-commit and related packages'
+name: 'Install prek'
+description: 'Installs prek and related packages'
 inputs:
   python-version:
     description: 'Python version to use'
     default: "3.10"
   uv-version:
     description: 'uv version to use'
     default: "0.8.9"  # Keep this comment to allow automatic replacement of uv 
version
-  pre-commit-version:
-    description: 'pre-commit version to use'
-    default: "4.3.0"  # Keep this comment to allow automatic replacement of 
pre-commit version
-  pre-commit-uv-version:
-    description: 'pre-commit-uv version to use'
-    default: "4.1.4"  # Keep this comment to allow automatic replacement of 
pre-commit-uv version
-  skip-pre-commits:
-    description: "Skip some pre-commits from installation"
+  prek-version:
+    description: 'prek version to use'
+    default: "0.0.25"  # Keep this comment to allow automatic replacement of 
prek version
+  skip-prek-hooks:
+    description: "Skip some prek hooks from installation"
     default: ""
 runs:
   using: "composite"
   steps:
-    - name: Install pre-commit, uv, and pre-commit-uv
+    - name: Install prek and uv
       shell: bash
       env:
         UV_VERSION: ${{inputs.uv-version}}
-        PRE_COMMIT_VERSION: ${{inputs.pre-commit-version}}
-        PRE_COMMIT_UV_VERSION: ${{inputs.pre-commit-uv-version}}
-        SKIP: ${{ inputs.skip-pre-commits }}
+        PREK_VERSION: ${{inputs.prek-version}}
+        SKIP: ${{ inputs.skip-prek-hooks }}
       run: |
         pip install uv==${UV_VERSION} || true
-        uv tool install pre-commit==${PRE_COMMIT_VERSION} --with 
uv==${UV_VERSION} \
-        --with pre-commit-uv==${PRE_COMMIT_UV_VERSION}
+        uv tool install prek==${PREK_VERSION} --with uv==${UV_VERSION}

Review Comment:
   ```suggestion
           uv tool install prek==${PREK_VERSION}
   ```
   
   I think it's not necessary to install prek `--with uv` since uv is already 
installed and prek will reuse that.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to