Skip to content
Commit 52ced851 authored by Peter Wen's avatar Peter Wen Committed by Chromium LUCI CQ
Browse files

Infra: Minor cleanup

Reorder some starlark load statements according to lexicographical
order. It has no functional impact but makes the LSP happy.

Move keyword args without defaults above ones that have defaults, when
the file contains these errors the LSP is unable to successfully parse
the entire file. This has no functional difference as they are all
keyword args. It has the benefit of highlighting the required ones above
the optional ones.

Add module descriptions and switch a list comprehension arg to a more
readable one (avoids potential confusion between 1 and l, suggested by
LSP).

Bug: 329156223
Change-Id: I3b51027481aad80608f3994f0b290a7858d7227c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5366268


Auto-Submit: Peter Wen <wnwen@chromium.org>
Reviewed-by: default avatarGarrett Beaty <gbeaty@google.com>
Commit-Queue: Garrett Beaty <gbeaty@google.com>
Cr-Commit-Position: refs/heads/main@{#1271748}
parent d98829f3
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment