switch to 4 space indent
Leave the recipes/ code at 2 space to match the rest of the recipes
project in other repos.
Reformatted using:
files=( $(
git ls-tree -r --name-only HEAD | \
grep -Ev -e '^(third_party|recipes)/' | \
grep '\.py$';
git grep -l '#!/usr/bin/env.*python' | grep -v '\.py$'
) )
parallel ./yapf -i -- "${files[@]}"
~/chromiumos/chromite/contrib/reflow_overlong_comments "${files[@]}"
The files that still had strings that were too long were manually
reformatted because they were easy and only a few issues.
autoninja.py
clang_format.py
download_from_google_storage.py
fix_encoding.py
gclient_utils.py
git_cache.py
git_common.py
git_map_branches.py
git_reparent_branch.py
gn.py
my_activity.py
owners_finder.py
presubmit_canned_checks.py
reclient_helper.py
reclientreport.py
roll_dep.py
rustfmt.py
siso.py
split_cl.py
subcommand.py
subprocess2.py
swift_format.py
upload_to_google_storage.py
These files still had lines (strings) that were too long, so the pylint
warnings were suppressed with a TODO.
auth.py
gclient.py
gclient_eval.py
gclient_paths.py
gclient_scm.py
gerrit_util.py
git_cl.py
presubmit_canned_checks.py
presubmit_support.py
scm.py
Change-Id: Ia6535c4f2c48d46b589ec1e791dde6c6b2ea858f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/4836379
Commit-Queue: Josip Sokcevic <sokcevic@chromium.org>
Auto-Submit: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Josip Sokcevic <sokcevic@chromium.org>
diff --git a/rustfmt.py b/rustfmt.py
index aae33a1..65246ae 100755
--- a/rustfmt.py
+++ b/rustfmt.py
@@ -15,56 +15,56 @@
class NotFoundError(Exception):
- """A file could not be found."""
-
- def __init__(self, e):
- Exception.__init__(
- self, 'Problem while looking for rustfmt in Chromium source tree:\n'
- '%s' % e)
+ """A file could not be found."""
+ def __init__(self, e):
+ Exception.__init__(
+ self, 'Problem while looking for rustfmt in Chromium source tree:\n'
+ '%s' % e)
def FindRustfmtToolInChromiumTree():
- """Return a path to the rustfmt executable, or die trying."""
- chromium_src_path = gclient_paths.GetPrimarySolutionPath()
- if not chromium_src_path:
- raise NotFoundError(
- 'Could not find checkout in any parent of the current path.\n'
- 'Set CHROMIUM_BUILDTOOLS_PATH to use outside of a chromium checkout.')
+ """Return a path to the rustfmt executable, or die trying."""
+ chromium_src_path = gclient_paths.GetPrimarySolutionPath()
+ if not chromium_src_path:
+ raise NotFoundError(
+ 'Could not find checkout in any parent of the current path.\n'
+ 'Set CHROMIUM_BUILDTOOLS_PATH to use outside of a chromium '
+ 'checkout.')
- tool_path = os.path.join(chromium_src_path, 'third_party', 'rust-toolchain',
- 'bin', 'rustfmt' + gclient_paths.GetExeSuffix())
- if not os.path.exists(tool_path):
- raise NotFoundError('File does not exist: %s' % tool_path)
- return tool_path
+ tool_path = os.path.join(chromium_src_path, 'third_party', 'rust-toolchain',
+ 'bin', 'rustfmt' + gclient_paths.GetExeSuffix())
+ if not os.path.exists(tool_path):
+ raise NotFoundError('File does not exist: %s' % tool_path)
+ return tool_path
def IsRustfmtSupported():
- try:
- FindRustfmtToolInChromiumTree()
- return True
- except NotFoundError:
- return False
+ try:
+ FindRustfmtToolInChromiumTree()
+ return True
+ except NotFoundError:
+ return False
def main(args):
- try:
- tool = FindRustfmtToolInChromiumTree()
- except NotFoundError as e:
- sys.stderr.write("%s\n" % str(e))
- return 1
+ try:
+ tool = FindRustfmtToolInChromiumTree()
+ except NotFoundError as e:
+ sys.stderr.write("%s\n" % str(e))
+ return 1
- # Add some visibility to --help showing where the tool lives, since this
- # redirection can be a little opaque.
- help_syntax = ('-h', '--help', '-help', '-help-list', '--help-list')
- if any(match in args for match in help_syntax):
- print('\nDepot tools redirects you to the rustfmt at:\n %s\n' % tool)
+ # Add some visibility to --help showing where the tool lives, since this
+ # redirection can be a little opaque.
+ help_syntax = ('-h', '--help', '-help', '-help-list', '--help-list')
+ if any(match in args for match in help_syntax):
+ print('\nDepot tools redirects you to the rustfmt at:\n %s\n' % tool)
- return subprocess.call([tool] + args)
+ return subprocess.call([tool] + args)
if __name__ == '__main__':
- try:
- sys.exit(main(sys.argv[1:]))
- except KeyboardInterrupt:
- sys.stderr.write('interrupted\n')
- sys.exit(1)
+ try:
+ sys.exit(main(sys.argv[1:]))
+ except KeyboardInterrupt:
+ sys.stderr.write('interrupted\n')
+ sys.exit(1)