{Pylint} Fix implicit-str-concat (#30363)

Author: bebound

pylintrc

@@ -46,7 +46,6 @@ disable=
     # These rules were added in Pylint >= 2.12, disable them to avoid making retroactive change
     missing-timeout,
     superfluous-parens,
-    implicit-str-concat,
     unnecessary-dunder-call,
     # These rules were added in Pylint >= 3.2
     possibly-used-before-assignment,

src/azure-cli/azure/cli/command_modules/acs/_roleassignments.py

@@ -301,7 +301,7 @@ def ensure_cluster_identity_permission_on_kubelet_identity(cmd, cluster_identity
         if use_shared_identity():
             return
         raise UnauthorizedError(
-            "Could not grant Managed Identity Operator " "permission to cluster identity at scope {}".format(scope)
+            "Could not grant Managed Identity Operator permission to cluster identity at scope {}".format(scope)
         )
 
 

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

@@ -438,7 +438,7 @@ def load_arguments(self, _):
         c.argument('dapr_enable_api_logging', options_list=['--dapr-enable-api-logging', '--dal'], help="Enable/Disable API logging for the Dapr sidecar.", arg_type=get_three_state_flag(return_label=True))
         c.argument('workload_profile_name', help="The name of the workload profile to run the app on.", is_preview=True)
         c.argument('cpu', type=float, help="Required CPU in cores from 0.5 to 2.0.", is_preview=True)
-        c.argument('memory', help="Required momory from 1.0 to 4.0 ending with ""Gi"" e.g. 1.0Gi, ", is_preview=True)
+        c.argument('memory', help="Required momory from 1.0 to 4.0 ending with Gi e.g. 1.0Gi, ", is_preview=True)
 
     with self.argument_context('functionapp runtime config') as c:
         c.argument('runtime_version', help='The version of the functions runtime stack. Use "az functionapp list-flexconsumption-runtimes" to check supported runtimes and versions', is_preview=True)

src/azure-cli/azure/cli/command_modules/billing/manual/_params.py

@@ -65,7 +65,7 @@ def load_arguments(self, _):
             "invoice_name",
             options_list=["--name", "-n", "--invoice-name"],
             type=str,
-            help="The ID that " "uniquely identifies an invoice.",
+            help="The ID that uniquely identifies an invoice.",
         )
         c.argument(
             "by_subscription",
@@ -129,7 +129,7 @@ def load_arguments(self, _):
             "name",
             options_list=["--name", "-n"],
             type=str,
-            help="The ID that uniquely identifies a role " "assignment.",
+            help="The ID that uniquely identifies a role assignment.",
         )
         c.argument(
             "profile_name",

src/azure-cli/azure/cli/command_modules/cdn/custom/custom_rule_util.py

@@ -292,7 +292,7 @@ def create_action(action_name, cache_behavior=None, cache_duration=None, header_
         action = {
             "cache_key_query_string": {
                 "parameters": {
-                    "type_name" "DeliveryRuleCacheKeyQueryStringBehaviorActionParameters"
+                    "type_name": "DeliveryRuleCacheKeyQueryStringBehaviorActionParameters",
                     "query_string_behavior": query_string_behavior,
                     "query_parameters": query_parameters
                 }

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

@@ -42,17 +42,17 @@ def load_arguments(self, _):
                 'namespace',
                 completer=get_providers_completion_list,
                 arg_group='Resource ID',
-                help='Provider namespace (Ex: ''Microsoft.Provider'').')
+                help='Provider namespace (Ex: Microsoft.Provider).')
             c.argument(
                 'resource_type_parent',
                 options_list=['--parent'],
                 arg_group='Resource ID',
-                help='The parent path (Ex: ''resourceTypeA/nameA/resourceTypeB/nameB'').')
+                help='The parent path (Ex: resourceTypeA/nameA/resourceTypeB/nameB).')
             c.argument(
                 'resource_type',
                 completer=get_resource_types_completion_list,
                 arg_group='Resource ID',
-                help='Resource type (Ex: ''resourceTypeC'').')
+                help='Resource type (Ex: resourceTypeC).')
 
     for scope in ['state', 'event']:
         with self.argument_context('policy {}'.format(scope)) as c:
@@ -137,7 +137,7 @@ def load_arguments(self, _):
             'location_filters',
             options_list='--location-filters',
             nargs='+',
-            help='Space separated list of resource locations that should be remediated (Ex: ''centralus westeurope'').')  # pylint: disable=line-too-long
+            help='Space separated list of resource locations that should be remediated (Ex: centralus westeurope).')  # pylint: disable=line-too-long
         c.argument(
             'policy_assignment',
             options_list=['--policy-assignment', '-a'],

src/azure-cli/azure/cli/command_modules/relay/tests/latest/test_relay_commands.py

@@ -102,7 +102,7 @@ def test_relay_hyco(self, resource_group):
             'loc': 'westus2',
             'rg': resource_group,
             'namespacename': self.create_random_name(prefix='relay-nscli', length=20),
-            'tags': 'tag1=value1' 'tag2=value2',
+            'tags': 'tag1=value1 tag2=value2',
             'authoname': self.create_random_name(prefix='cliAutho', length=20),
             'defaultauthorizationrule': 'RootManageSharedAccessKey',
             'accessrights': 'Listen',
@@ -185,7 +185,7 @@ def test_relay_wcfrelay(self, resource_group):
             'rg': resource_group,
             'relaytype': 'Http',
             'namespacename': self.create_random_name(prefix='relay-nscli', length=20),
-            'tags': 'tag1=value1' 'tag2=value2',
+            'tags': 'tag1=value1 tag2=value2',
             'authoname': self.create_random_name(prefix='cliAutho', length=20),
             'defaultauthorizationrule': 'RootManageSharedAccessKey',
             'accessrights': 'Send',

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

@@ -48,7 +48,7 @@ def pre_operations(self):
 
         if args.hosting_mode == "highDensity" and args.sku != "standard3":
             raise UnrecognizedArgumentError(
-                "SearchService.HostingMode: ""highDensity"" is only allowed when sku is ""standard3""")
+                "SearchService.HostingMode: highDensity is only allowed when sku is standard3")
 
         if has_value(args.ip_rules):
             ip_rules = re.split(';|,', args.ip_rules.to_serialized_data())
@@ -173,7 +173,7 @@ def update_search_service(instance, partition_count=0, replica_count=0, public_n
     if public_network_access:
         if (public_network_access.lower() not in ["enabled", "disabled"]):
             raise UnrecognizedArgumentError(
-                "SearchService.PublicNetworkAccess: only [""enabled"", ""disabled""] are allowed")
+                "SearchService.PublicNetworkAccess: only [enabled, disabled] are allowed")
         instance.public_network_access = public_network_access
     if ip_rules:
         _ip_rules = []
@@ -318,12 +318,12 @@ def setup_search_auth(instance, disable_local_auth, auth_options, aad_auth_failu
     # Done by argument define
     if (auth_options is not None and auth_options not in ["aadOrApiKey", "apiKeyOnly"]):
         raise UnrecognizedArgumentError(
-            "SearchService.AuthOptions: only [""aadOrApiKey"", ""apiKeyOnly""] are allowed")
+            "SearchService.AuthOptions: only [aadOrApiKey, apiKeyOnly] are allowed")
     # Done in aaz by default
     if (aad_auth_failure_mode is not None and aad_auth_failure_mode not in ["http401WithBearerChallenge", "http403"]):
         raise UnrecognizedArgumentError(
             "SearchService.AuthOptions.AadAuthFailureMode: only "
-            "[""http401WithBearerChallenge"", ""http403""] are allowed")
+            "[http401WithBearerChallenge, http403] are allowed")
 
     # Done in pre_operations
     if disable_local_auth and auth_options:

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

@@ -281,7 +281,7 @@ def _find_edition_capability(sku, supported_editions):
             return next(e for e in supported_editions if e.name == sku.tier)
         except StopIteration:
             candidate_editions = [e.name for e in supported_editions]
-            raise CLIError('Could not find tier ''{}''. Supported tiers are: {}'.format(
+            raise CLIError('Could not find tier {}. Supported tiers are: {}'.format(
                 sku.tier, candidate_editions
             ))
     else:
@@ -304,7 +304,7 @@ def _find_family_capability(sku, supported_families):
             return next(f for f in supported_families if f.name == sku.family)
         except StopIteration:
             candidate_families = [e.name for e in supported_families]
-            raise CLIError('Could not find family ''{}''. Supported families are: {}'.format(
+            raise CLIError('Could not find family {}. Supported families are: {}'.format(
                 sku.family, candidate_families
             ))
     else:

src/azure-cli/azure/cli/command_modules/storage/commands.py

@@ -392,7 +392,7 @@ def get_custom_sdk(custom_module, client_factory, resource_type=ResourceType.DAT
         g.storage_custom_command_oauth('generate-sas', 'generate_sas_blob_uri')
 
     blob_service_sdk = CliCommandType(
-        operations_tmpl='azure.multiapi.storagev2.blob._blob_service_client#' 'BlobServiceClient.{}',
+        operations_tmpl='azure.multiapi.storagev2.blob._blob_service_client#BlobServiceClient.{}',
         client_factory=cf_blob_service,
         resource_type=ResourceType.DATA_STORAGE_BLOB
     )