scripts: Lint fixes.
line-too-long and docstring-section-indent fixes. Misc spelling
and grammar fixes.
BUG=None
TEST=CQ
Change-Id: If3f29b7a12425947e4fa626cf5a403229ebc2885
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4429332
Tested-by: Alex Klein <saklein@chromium.org>
Auto-Submit: Alex Klein <saklein@chromium.org>
Reviewed-by: Cindy Lin <xcl@google.com>
Commit-Queue: Alex Klein <saklein@chromium.org>
diff --git a/scripts/loman.py b/scripts/loman.py
index 56bbffe..bb0527b 100644
--- a/scripts/loman.py
+++ b/scripts/loman.py
@@ -15,7 +15,7 @@
class LocalManifest(object):
- """Class which provides an abstraction for manipulating the local manifest."""
+ """Abstraction for manipulating the local manifest."""
@classmethod
def FromPath(cls, path, empty_if_missing=False):
@@ -44,7 +44,7 @@
"""Accessor method for getting a project node from the manifest tree.
Returns:
- project element node from ElementTree, otherwise, None
+ project element node from ElementTree, otherwise, None
"""
if path is None:
# Use a unique value that can't ever match.
@@ -88,7 +88,7 @@
# Per repo behaviour, enforce an appropriate platform group if
# we're converting from a default manifest group to a limited one.
# Finally, note we reprocess the existing groups; this is to allow
- # us to cleanup any user screwups, or our own screwups.
+ # us to clean up any user screwups, or our own screwups.
requested_groups = (
["minilayout", "platform-%s" % (platform.system().lower(),)]
+ enabled_groups
@@ -210,9 +210,11 @@
elif main_element:
if options.remote is not None:
- # Likely this project wasn't meant to be remote, so workon main element
+ # Likely this project wasn't meant to be remote, so workon main
+ # element.
print(
- "Project already exists in manifest. Using that as workon project."
+ "Project already exists in manifest. "
+ "Using that as workon project."
)
_AddProjectsToManifestGroups(
options, [checkout["name"] for checkout in main_element]