{Zones} Release bugfix to improve command loading (Azure#8766)

Author: nielsams

src/zones/HISTORY.rst

@@ -3,6 +3,10 @@
 Release History
 ===============
 
+1.0.0b3
+++++++
+* Minor bugfixes to improve command loading
+
 1.0.0b2
 ++++++
 * Minor bugfixes to improve command loading

src/zones/azext_zones/resource_type_validators/__init__.py

@@ -0,0 +1,5 @@
+# --------------------------------------------------------------------------------------------
+# Copyright (c) Microsoft Corporation. All rights reserved.
+# Licensed under the MIT License. See License.txt in the project root for license information.
+# --------------------------------------------------------------------------------------------
+"""Validators for resource types supported by Zones"""

src/zones/azext_zones/resource_type_validators/microsoft_apimanagement.py

@@ -10,6 +10,7 @@
 from knack.log import get_logger
 
 
+# pylint: disable=too-few-public-methods
 @register_resource_type("microsoft.apimanagement")
 class microsoft_apimanagement:
     @staticmethod
@@ -33,7 +34,6 @@ def validate(resource):
             zones = resource.get("zones") or []
             if len(zones) > 1 and resource["sku"]["name"] == "Premium":
                 return ZoneRedundancyValidationResult.Yes
-            else:
-                return ZoneRedundancyValidationResult.No
+            return ZoneRedundancyValidationResult.No
 
         return ZoneRedundancyValidationResult.Unknown

src/zones/azext_zones/resource_type_validators/microsoft_app.py

@@ -10,6 +10,7 @@
 from knack.log import get_logger
 
 
+# pylint: disable=too-few-public-methods
 @register_resource_type("microsoft.app")
 class microsoft_app:
     @staticmethod
@@ -32,7 +33,6 @@ def validate(resource):
             # https://learn.microsoft.com/azure/reliability/reliability-azure-container-apps#availability-zone-support
             if resource["properties"].get("zoneRedundant", {}) is True:
                 return ZoneRedundancyValidationResult.Yes
-            else:
-                return ZoneRedundancyValidationResult.No
+            return ZoneRedundancyValidationResult.No
 
         return ZoneRedundancyValidationResult.Unknown

src/zones/azext_zones/resource_type_validators/microsoft_authorization.py

@@ -10,6 +10,7 @@
 from knack.log import get_logger
 
 
+# pylint: disable=too-few-public-methods
 @register_resource_type("microsoft.authorization")
 class microsoft_authorization:
     @staticmethod

src/zones/azext_zones/resource_type_validators/microsoft_automation.py

@@ -10,6 +10,7 @@
 from knack.log import get_logger
 
 
+# pylint: disable=too-few-public-methods
 @register_resource_type("microsoft.automation")
 class microsoft_automation:
     @staticmethod

src/zones/azext_zones/resource_type_validators/microsoft_botservice.py

@@ -10,6 +10,7 @@
 from knack.log import get_logger
 
 
+# pylint: disable=too-few-public-methods
 @register_resource_type("microsoft.botservice")
 class microsoft_botservice:
     @staticmethod
@@ -32,8 +33,8 @@ def validate(resource):
                     "Your bot service resource in westeurope may be zone redundant, \
                     but only if it's configured as a regional (not global) bot. Please check manually."
                 )
-            else:
-                # Bot services cannot be ZR in any other region
-                return ZoneRedundancyValidationResult.No
+                return ZoneRedundancyValidationResult.Unknown
+            # Bot services cannot be ZR in any other region
+            return ZoneRedundancyValidationResult.No
 
         return ZoneRedundancyValidationResult.Unknown

src/zones/azext_zones/resource_type_validators/microsoft_cache.py

@@ -10,6 +10,7 @@
 from knack.log import get_logger
 
 
+# pylint: disable=too-few-public-methods
 @register_resource_type("microsoft.cache")
 class microsoft_cache:
     @staticmethod
@@ -27,15 +28,13 @@ def validate(resource):
             zones = resource.get("zones") or []
             if len(zones) > 1:
                 return ZoneRedundancyValidationResult.Yes
-            else:
-                return ZoneRedundancyValidationResult.No
+            return ZoneRedundancyValidationResult.No
 
         # Redis Enterprise
         if resourceSubType == "redisenterprise":
             zones = resource.get("zones") or []
             if len(zones) > 1:
                 return ZoneRedundancyValidationResult.Yes
-            else:
-                return ZoneRedundancyValidationResult.No
+            return ZoneRedundancyValidationResult.No
 
         return ZoneRedundancyValidationResult.Unknown

src/zones/azext_zones/resource_type_validators/microsoft_cdn.py

@@ -10,6 +10,7 @@
 from knack.log import get_logger
 
 
+# pylint: disable=too-few-public-methods
 @register_resource_type("microsoft.cdn")
 class microsoft_cdn:
     @staticmethod

src/zones/azext_zones/resource_type_validators/microsoft_chaos.py

@@ -10,6 +10,7 @@
 from knack.log import get_logger
 
 
+# pylint: disable=too-few-public-methods
 @register_resource_type("microsoft.chaos")
 class microsoft_chaos:
     @staticmethod