commit e7fa5648e49726f68061a6d839d45657436f87dc
author Matt Stark <msta@google.com> Tue May 02 11:14:27 2023 +1000
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu May 04 17:51:12 2023 +0000
tree 38b42fef9de708da5c239b9afbf0fcb31675d885
parent 7b815f09612e8a9ad0a4d8330b9f6b1f2e728dcf

Remove line limit for bzl files.

BUG=None
TEST=repo upload on commit with >80 line length in bzl files.

Change-Id: Ib83bb4199efb60d5eaa9a502b99db07c6e93ca7a
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/repohooks/+/4494674
Tested-by: Matt Stark <msta@google.com>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Auto-Submit: Matt Stark <msta@google.com>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

pre-upload.py

diff --git a/pre-upload.py b/pre-upload.py
index 85d47ee..bef03f1 100755
--- a/pre-upload.py
+++ b/pre-upload.py
@@ -681,6 +681,11 @@
         # "it should not be enforced in code reviews or presubmit scripts".
         # https://bazel.build/build/style-guide?hl=en
         r".*\.bazel$",
+        # "As in BUILD files, there is no strict line length limit as labels
+        # can be long. When possible, try to use at most 79 characters per line
+        # (following Python's style guide, PEP 8)."
+        # https://bazel.build/rules/bzl-style#line-length
+        r".*\.bzl$",
         # GN format does not enforce a line length limit by the cros format.
         # https://chromium.googlesource.com/dart/dartium/src/+/HEAD/tools/gn/docs/style_guide.md
         r".*\.gn$",