Fix their style and linter

src/azure-cli/azure/cli/command_modules/batch/_params.py

@@ -88,7 +88,7 @@ def load_arguments(self, _):
         c.argument('account_name', batch_name_type, options_list=('--name', '-n'),
                    help='Name of the batch account to show. If not specified will display currently set account.',
                    required=False)
-        c.argument('key_name', arg_type=get_enum_type(AccountKeyType))
+        c.argument('key_name', arg_type=get_enum_type(AccountKeyType), help='Name of the batch account key.')
 
     with self.argument_context('batch account login') as c:
         c.argument('shared_key_auth', action='store_true', help='Using Shared Key authentication, if not specified, it will use Azure Active Directory authentication.')

src/azure-cli/azure/cli/command_modules/batch/custom.py

@@ -176,7 +176,7 @@ def renew_accounts_keys(client, resource_group_name, account_name, key_name=None
     parameters = BatchAccountRegenerateKeyParameters(key_name=key_name)
 
     return client.regenerate_key(resource_group_name=resource_group_name,
-                       account_name=account_name,parameters=parameters)
+                                 account_name=account_name, parameters=parameters)
 
 
 @transfer_doc(Application)
@@ -241,12 +241,8 @@ def create_application_package(cmd, client,
     return client.get(resource_group_name, account_name, application_name, version_name)
 
 
-
 @transfer_doc(ApplicationPackageOperations.activate)
-def activate_application_package(cmd, client,
-                               resource_group_name, account_name, application_name, version_name,
-                                 format):
-
+def activate_application_package(client, resource_group_name, account_name, application_name, version_name, format):
     # activate the application package
     parameters = ActivateApplicationPackageParameters(format=format)
     return client.activate(resource_group_name, account_name, application_name, version_name, parameters)