git cl: style cleanup.

git_cl.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Copyright (C) 2008 Evan Martin <martine@danga.com>
 
 """A git-command for integrating reviews on Rietveld and Gerrit."""
 
 from __future__ import print_function
@@ skipping 4589 matching lines @@
 
   print('The tree is %s' % status)
   print()
   print(GetTreeStatusReason())
   if status != 'open':
     return 1
   return 0
 
 
 def CMDtry(parser, args):
-  """Triggers try jobs through BuildBucket."""
-  group = optparse.OptionGroup(parser, "Try job options")
+  '''Triggers try jobs through BuildBucket.'''
+  group = optparse.OptionGroup(parser, 'Try job options')
   group.add_option(
-      "-b", "--bot", action="append",
-      help=("IMPORTANT: specify ONE builder per --bot flag. Use it multiple "
-            "times to specify multiple builders. ex: "
-            "'-b win_rel -b win_layout'. See "
-            "the try server waterfall for the builders name and the tests "
-            "available."))
+      '-b', '--bot', action='append',
+      help=('IMPORTANT: specify ONE builder per --bot flag. Use it multiple '
+            'times to specify multiple builders. ex: '
+            '"-b win_rel -b win_layout". See '
+            'the try server waterfall for the builders name and the tests '
+            'available.'))
   group.add_option(
-      "-m", "--master", default='',
-      help=("Specify a try master where to run the tries."))
+      '-m', '--master', default='',
+      help=('Specify a try master where to run the tries.'))
   group.add_option(
-      "-r", "--revision",
-      help="Revision to use for the try job; default: the "
-           "revision will be determined by the try server; see "
-           "its waterfall for more info")
+      '-r', '--revision',
+      help='Revision to use for the try job; default: the '
+           'revision will be determined by the try server; see '
+           'its waterfall for more info')
   group.add_option(
-      "-c", "--clobber", action="store_true", default=False,
-      help="Force a clobber before building; e.g. don't do an "
-           "incremental build")
+      '-c', '--clobber', action='store_true', default=False,
+      help='Force a clobber before building; e.g. don\'t do an '
+           'incremental build')
   group.add_option(
-      "--project",
-      help="Override which project to use. Projects are defined "
-           "server-side to define what default bot set to use")
+      '--project',
+      help='Override which project to use. Projects are defined '
+           'server-side to define what default bot set to use')
   group.add_option(
-      "-p", "--property", dest="properties", action="append", default=[],
-      help="Specify generic properties in the form -p key1=value1 -p "
-           "key2=value2 etc (buildbucket only). The value will be treated as "
-           "json if decodable, or as string otherwise.")
+      '-p', '--property', dest='properties', action='append', default=[],
+      help='Specify generic properties in the form -p key1=value1 -p '
+           'key2=value2 etc (buildbucket only). The value will be treated as '
+           'json if decodable, or as string otherwise.')
   group.add_option(
-      "-n", "--name", help="Try job name; default to current branch name")
+      '-n', '--name', help='Try job name; default to current branch name')
   group.add_option(
-      "--use-rietveld", action="store_true", default=False,
-      help="Use Rietveld to trigger try jobs.")
+      '--use-rietveld', action='store_true', default=False,
+      help='Use Rietveld to trigger try jobs.')
   group.add_option(
-      "--buildbucket-host", default='cr-buildbucket.appspot.com',
-      help="Host of buildbucket. The default host is %default.")
+      '--buildbucket-host', default='cr-buildbucket.appspot.com',
+      help='Host of buildbucket. The default host is %default.')
   parser.add_option_group(group)
   auth.add_auth_options(parser)
   options, args = parser.parse_args(args)
   auth_config = auth.extract_auth_config_from_options(options)
 
   if options.use_rietveld and options.properties:
     parser.error('Properties can only be specified with buildbucket')
 
   # Make sure that all properties are prop=value pairs.
   bad_params = [x for x in options.properties if '=' not in x]
@@ skipping 148 matching lines @@
     for (master, builders) in sorted(masters.iteritems()):
       if master:
         print('Master: %s' % master)
       length = max(len(builder) for builder in builders)
       for builder in sorted(builders):
         print('  %*s: %s' % (length, builder, ','.join(builders[builder])))
   return 0
 
 
 def CMDtry_results(parser, args):
-  group = optparse.OptionGroup(parser, "Try job results options")
+  """Prints info about try jobs associated with current CL."""
+  group = optparse.OptionGroup(parser, 'Try job results options')
   group.add_option(
-      "-p", "--patchset", type=int, help="patchset number if not current.")
+      '-p', '--patchset', type=int, help='patchset number if not current.')
   group.add_option(
-      "--print-master", action='store_true', help="print master name as well.")
+      '--print-master', action='store_true', help='print master name as well.')
   group.add_option(
-      "--color", action='store_true', default=setup_color.IS_TTY,
-      help="force color output, useful when piping output.")
+      '--color', action='store_true', default=setup_color.IS_TTY,
+      help='force color output, useful when piping output.')
   group.add_option(
-      "--buildbucket-host", default='cr-buildbucket.appspot.com',
-      help="Host of buildbucket. The default host is %default.")
+      '--buildbucket-host', default='cr-buildbucket.appspot.com',
+      help='Host of buildbucket. The default host is %default.')
   group.add_option(
       '--json', help='Path of JSON output file to write try job results to.')
   parser.add_option_group(group)
   auth.add_auth_options(parser)
   options, args = parser.parse_args(args)
   if args:
     parser.error('Unrecognized args: %s' % ' '.join(args))
 
   auth_config = auth.extract_auth_config_from_options(options)
   cl = Changelist(auth_config=auth_config)
@@ skipping 462 matching lines @@
 if __name__ == '__main__':
   # These affect sys.stdout so do it outside of main() to simplify mocks in
   # unit testing.
   fix_encoding.fix_encoding()
   setup_color.init()
   try:
     sys.exit(main(sys.argv[1:]))
   except KeyboardInterrupt:
     sys.stderr.write('interrupted\n')
     sys.exit(1)