Run ruff pre-commit rule in CI

Closes: ray-project#48508 Signed-off-by: Chi-Sheng Liu <[email protected]>
MortalHappiness · Dec 28, 2024 · 513e7a1 · 513e7a1
1 parent ef84628
commit 513e7a1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/.buildkite/lint.rayci.yml b/.buildkite/lint.rayci.yml
@@ -9,6 +9,7 @@ steps:
     matrix:
       - clang_format
       - code_format
+      - pre_commit
       - untested_code_snippet
       - banned_words
       - doc_readme

diff --git a/ci/lint/lint.sh b/ci/lint/lint.sh
@@ -10,6 +10,14 @@ clang_format() {
   ./ci/lint/check-git-clang-format-output.sh
 }
 
+pre_commit() {
+  # Run pre-commit on all files
+  # TODO(MortalHappiness): Run all pre-commit checks
+  # Currently, we only run the ruff format check
+  pip install -c python/requirements_compiled.txt pre-commit
+  pre-commit run ruff --all-files
+}
+
 code_format() {
   pip install -c python/requirements_compiled.txt -r python/requirements/lint-requirements.txt
   FORMAT_SH_PRINT_DIFF=1 ./ci/lint/format.sh --all-scripts