Skip to content

Commit 1d1b1b0

Browse files
authored
[ci] bazel lint all BUILD files - python/ (#51092)
Signed-off-by: Elijah Melton <elimelt@uw.edu>
1 parent 894bd0b commit 1d1b1b0

File tree

27 files changed

+3474
-1405
lines changed

27 files changed

+3474
-1405
lines changed

.pre-commit-config.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,9 @@ repos:
5353
rev: 8.0.1
5454
hooks:
5555
- id: buildifier
56-
files: ^(src|cpp)(/[^/]+)*/BUILD$
56+
files: ^(src|cpp|python)(/[^/]+)*/BUILD$
5757
- id: buildifier-lint
58-
files: ^(src|cpp)(/[^/]+)*/BUILD$
58+
files: ^(src|cpp|python)(/[^/]+)*/BUILD$
5959

6060
- repo: https://github.com/psf/black
6161
rev: 22.10.0

python/ray/_private/BUILD

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
load("//bazel:python.bzl", "doctest")
22

33
doctest(
4-
files = glob(["**/*.py"], exclude=["**/thirdparty_files/**"]),
5-
tags = ["team:core"]
4+
files = glob(
5+
["**/*.py"],
6+
exclude = ["**/thirdparty_files/**"],
7+
),
8+
tags = ["team:core"],
69
)

python/ray/_private/runtime_env/BUILD

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# TODO(hjiang): All existing pythons are not using bazel as build system, which leads to missing BUILD file and targets.
22
# Revisit if we decide to support bazel build in the future.
33

4-
load("@rules_python//python:defs.bzl", "py_library", "py_test")
4+
load("@rules_python//python:defs.bzl", "py_library")
55

66
package(default_visibility = ["//visibility:public"])
77

@@ -33,10 +33,10 @@ py_library(
3333

3434
py_library(
3535
name = "uv",
36-
srcs= ["uv.py"],
36+
srcs = ["uv.py"],
3737
deps = [
38+
":dependency_utils",
3839
":utils",
3940
":virtualenv_utils",
40-
":dependency_utils",
4141
],
4242
)

0 commit comments

Comments
 (0)