Issue 5844 - Remove redundant parentheses in buildtools

build.py

 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 import argparse
 import logging
 import os
 import re
 import shutil
 import subprocess
@@ skipping 109 matching lines @@
 
 
 build_available_subcommands._result = None
 
 
 @argparse_command(
     valid_platforms={'chrome', 'gecko', 'edge'},
     arguments=(
         make_argument(
             '-b', '--build-num', dest='build_num',
-            help=('Use given build number (if omitted the build number will '
-                  'be retrieved from Mercurial)')),
+            help='Use given build number (if omitted the build number will '
+                  'be retrieved from Mercurial)'),

[Sebastian Noack, 2018/01/17 00:44:54]

The indentation is off by one. Please align subsequent lines, so that the
opening quotes are on the same column as in the first line. Same below.

[rosie, 2018/01/25 19:22:19]

Done.

         make_argument(
             '-k', '--key', dest='key_file',
-            help=('File containing private key and certificates required to '
-                  'sign the package')),
+            help='File containing private key and certificates required to '
+                  'sign the package'),
         make_argument(
             '-r', '--release', action='store_true',
             help='Create a release build'),
         make_argument('output_file', nargs='?')
     )
 )
 def build(base_dir, build_num, key_file, release, output_file, platform,
           **kwargs):
     """
     Create a build.
@@ skipping 177 matching lines @@
         raise argparse.ArgumentTypeError('Wrong version number format')
 
     return value
 
 
 @argparse_command(
     valid_platforms={'chrome', 'gecko', 'edge'}, multi_platform=True,
     arguments=(
         make_argument(
             '-k', '--key', dest='key_file',
-            help=('File containing private key and certificates required to '
-                  'sign the release.')),
+            help='File containing private key and certificates required to '
+                  'sign the release.'),
         make_argument(
             '-d', '--downloads-repository', dest='downloads_repository',
-            help=('Directory containing downloads repository (if omitted '
-                  '../downloads is assumed)')),
+            help='Directory containing downloads repository (if omitted '
+                  '../downloads is assumed)'),
         make_argument(
             'version', help='Version number of the release',
             type=valid_version_format)
     )
 )
 def release(base_dir, downloads_repository, key_file, platform, version,
             **kwargs):
     """
     Run release automation.
 
@@ skipping 30 matching lines @@
     if build_available_subcommands(base_dir):
         MAIN_PARSER.set_defaults(base_dir=base_dir)
 
         # If no args are provided, this module is run directly from the command
         # line. argparse will take care of consuming sys.argv.
         arguments = MAIN_PARSER.parse_args(args if len(args) > 0 else None)
 
         function = arguments.function
         del arguments.function
         function(**vars(arguments))