commit ab87cebda9434790247fc2219570190f70cd7c47
author Alex Klein <saklein@chromium.org> Tue Jan 24 12:00:51 2023 -0700
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jan 25 23:08:12 2023 +0000
tree 96b0f556ef4a1ee8052e21f784091bf348b70342
parent 6e10252ea5b302b747a5c3baa6ced07d5acad051

api: controller: Lint fixes.

Fix line-too-long and docstring-section-indent errors.

BUG=b:233893248
TEST=cros lint

Change-Id: I52ac19eb3a38066d4d09d56d27d61c904269a0e7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4190794
Tested-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Alex Klein <saklein@chromium.org>
Auto-Submit: Alex Klein <saklein@chromium.org>
Reviewed-by: Cindy Lin <xcl@google.com>

api/controller/sysroot.py

diff --git a/api/controller/sysroot.py b/api/controller/sysroot.py
index e65123a..00222b0 100644
--- a/api/controller/sysroot.py
+++ b/api/controller/sysroot.py
@@ -65,7 +65,9 @@
                 artifact_type=cabs.Sysroot.ArtifactType.SIMPLE_CHROME_SYSROOT,
                 paths=[
                     common_pb2.Path(
-                        path="/tmp/sysroot_chromeos-base_chromeos-chrome.tar.xz",
+                        path=(
+                            "/tmp/sysroot_chromeos-base_chromeos-chrome.tar.xz"
+                        ),
                         location=common_pb2.Path.OUTSIDE,
                     )
                 ],
@@ -100,8 +102,8 @@
 ) -> list:
     """Builds and copies sysroot artifacts to specified output_dir.
 
-    Copies sysroot artifacts to output_dir, returning a list of (output_dir: str)
-    paths to the desired files.
+    Copies sysroot artifacts to output_dir, returning a list of
+    (output_dir: str) paths to the desired files.
 
     Args:
         in_proto: Proto request defining reqs.
@@ -114,6 +116,7 @@
         A list of dictionary mappings of ArtifactType to list of paths.
     """
     generated = []
+    # pylint: disable=line-too-long
     artifact_types = {
         in_proto.ArtifactType.SIMPLE_CHROME_SYSROOT: sysroot.CreateSimpleChromeSysroot,
         in_proto.ArtifactType.CHROME_EBUILD_ENV: sysroot.CreateChromeEbuildEnv,
@@ -121,6 +124,7 @@
         in_proto.ArtifactType.DEBUG_SYMBOLS: sysroot.BundleDebugSymbols,
         in_proto.ArtifactType.FUZZER_SYSROOT: sysroot.CreateFuzzerSysroot,
     }
+    # pylint: enable=line-too-long
 
     for output_artifact in in_proto.output_artifacts:
         for artifact_type, func in artifact_types.items():
@@ -309,7 +313,8 @@
         for x in input_proto.package_indexes
     ]
 
-    # Calculate which packages would have been merged, but don't install anything.
+    # Calculate which packages would have been merged, but don't install
+    # anything.
     dryrun = input_proto.flags.dryrun
 
     if not target_sysroot.IsToolchainInstalled():