MyPy, PyLint, Snippets

Author: Nicola Camillucci

sdk/keyvault/azure-keyvault-administration/README.md

@@ -193,8 +193,7 @@ role_definition = client.set_role_definition(scope=scope, role_name=role_name, p
 ```python
 new_permissions = [
     KeyVaultPermission(
-        data_actions=[KeyVaultDataAction.READ_HSM_KEY],
-        not_data_actions=[KeyVaultDataAction.CREATE_HSM_KEY]
+        data_actions=[KeyVaultDataAction.READ_HSM_KEY], not_data_actions=[KeyVaultDataAction.CREATE_HSM_KEY]
     )
 ]
 unique_definition_name = role_definition.name

sdk/keyvault/azure-keyvault-keys/azure/keyvault/keys/_models.py

@@ -253,11 +253,11 @@ def recovery_level(self) -> Optional[str]:
         return self._attributes.recovery_level if self._attributes else None
 
     @property
-    def tags(self) -> Dict[str, str]:
+    def tags(self) -> Optional[Dict[str, str]]:
         """Application specific metadata in the form of key-value pairs.
 
         :returns: A dictionary of tags attached to the key.
-        :rtype: dict[str, str]
+        :rtype: dict[str, str] or None
         """
         return self._tags
 
@@ -313,9 +313,8 @@ def attestation(self) -> Optional[KeyAttestation]:
         # attestation was added in 7.6-preview.2
         if self._attributes:
             attestation = getattr(self._attributes, "attestation", None)
-            return (
-                KeyAttestation._from_generated(attestation=attestation) if attestation else None
-            )  # pylint:disable=protected-access
+            if attestation:
+                return KeyAttestation._from_generated(attestation=attestation)  # pylint:disable=protected-access
         return None
 
 
@@ -414,8 +413,8 @@ def _from_generated(cls, policy: "_models.KeyRotationPolicy") -> "KeyRotationPol
             []
             if policy.lifetime_actions is None
             else [
-                KeyRotationLifetimeAction._from_generated(action)
-                for action in policy.lifetime_actions  # pylint:disable=protected-access
+                KeyRotationLifetimeAction._from_generated(action)  # pylint:disable=protected-access
+                for action in policy.lifetime_actions
             ]
         )
         if policy.attributes:

sdk/keyvault/azure-keyvault-keys/azure/keyvault/keys/crypto/_models.py

@@ -240,8 +240,11 @@ def verify(
         if not result.is_valid:
             raise InvalidSignature(f"The provided signature '{signature!r}' is invalid.")
 
-    def recover_data_from_signature(
-        self, signature: bytes, padding: AsymmetricPadding, algorithm: Optional[HashAlgorithm]
+    def recover_data_from_signature(  # type: ignore[override]  # Parameter subset
+        self,
+        signature: bytes,
+        padding: AsymmetricPadding,
+        algorithm: Optional[HashAlgorithm],
     ) -> bytes:
         # pylint: disable=line-too-long
         """Recovers the signed data from the signature. Only supported with `cryptography` version 3.3 and above.
@@ -392,7 +395,7 @@ def public_key(self) -> KeyVaultRSAPublicKey:
         """
         return KeyVaultRSAPublicKey(self._client, self._key)
 
-    def sign(
+    def sign(  # type: ignore[override]  # Parameter subset
         self,
         data: bytes,
         padding: AsymmetricPadding,
@@ -413,8 +416,8 @@ def sign(
         :returns: The signature, as bytes.
         :rtype: bytes
         """
-        if isinstance(algorithm, Prehashed):
-            raise ValueError("`Prehashed` algorithms are unsupported. Please provide a `HashAlgorithm` instead.")
+        if not isinstance(algorithm, HashAlgorithm):
+            raise ValueError("Only `HashAlgorithm`s are supported. Please provide a `HashAlgorithm` instead.")
         mapped_algorithm = get_signature_algorithm(padding, algorithm)
         digest = Hash(algorithm)
         digest.update(data)

sdk/keyvault/azure-keyvault-keys/pyproject.toml

@@ -6,7 +6,7 @@
 # --------------------------------------------------------------------------
 
 [build-system]
-requires = ["setuptools>=61.0.0", "wheel"]  # Requires 61.0.0 for dynamic version
+requires = ["setuptools>=77.0.3", "wheel"]
 build-backend = "setuptools.build_meta"
 
 [project]
@@ -44,7 +44,7 @@ dynamic = [
 repository = "https://github.com/Azure/azure-sdk-for-python"
 
 [tool.setuptools.dynamic]
-version = {attr = "azure.keyvault.keys._version.VERSION"}
+version = {attr = "azure.keyvault.keys._generated._version.VERSION"}
 readme = {file = ["README.md", "CHANGELOG.md"], content-type = "text/markdown"}
 
 [tool.setuptools.packages.find]
@@ -56,6 +56,7 @@ exclude = [
     "doc*",
     "azure",
     "azure.keyvault",
+    "azure.keyvault.keys",
 ]
 
 [tool.setuptools.package-data]
@@ -64,23 +65,6 @@ pytyped = ["py.typed"]
 [tool.azure-sdk-build]
 pyright = false
 
-[tool.uv.sources]
-azure-core = { path = "../../core/azure-core" }
-azure-keyvault-nspkg = { path = "../../nspkg/azure-keyvault-nspkg" }
-azure-sdk-tools = { path = "../../../eng/tools/azure-sdk-tools" }
-
-[dependency-groups]
-dev = [
-    "aiohttp>=3.0",
-    "azure-core",
-    "azure-identity>=1.24.0",
-    "azure-keyvault-nspkg",
-    "azure-mgmt-keyvault==10.1.0",
-    "azure-sdk-tools",
-    "parameterized>=0.7.3",
-    "python-dateutil>=2.8.0",
-]
-
 [tool.azure-sdk-conda]
 in_bundle = true
 bundle_name = "azure-keyvault"