-
Notifications
You must be signed in to change notification settings - Fork 4.1k
[Dev][CI] Run pre-commit to all files #40542
Copy link
Copy link
Closed
Milestone
Description
Describe the enhancement requested
All checks are skipped now:
https://github.com/apache/arrow/actions/runs/8277176289/job/22647020698#step:6:12
Release Audit Tool.......................................................Passed
Docker Format........................................(no files to check)Skipped
Python Format........................................(no files to check)Skipped
cython-lint..........................................(no files to check)Skipped
C/GLib Format........................................(no files to check)Skipped
It seems that we need to use --all-files option.
https://pre-commit.com/#usage-in-continuous-integration
pre-commit can also be used as a tool for continuous integration. For instance, adding
pre-commit run --all-filesas a CI step will ensure everything stays in tip-top shape. To check only files which have changed, which may be faster, use something likepre-commit run --from-ref origin/HEAD --to-ref HEAD
Component(s)
Continuous Integration, Developer Tools
Reactions are currently unavailable