Skip to content

Commit 63d21c5

Browse files
author
Yalin Li
authored
Run mypy in azure-keyvault-certificates CI (Azure#20525)
1 parent 10979de commit 63d21c5

File tree

7 files changed

+154
-123
lines changed

7 files changed

+154
-123
lines changed

eng/tox/mypy_hard_failure_packages.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
"azure-eventhub",
1111
"azure-identity",
1212
"azure-keyvault-administration",
13+
"azure-keyvault-certificates",
1314
"azure-servicebus",
1415
"azure-ai-textanalytics",
1516
"azure-ai-formrecognizer",

sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_client.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ def begin_create_certificate(self, certificate_name, policy, **kwargs):
134134
create_certificate_polling = CreateCertificatePoller(
135135
get_certificate_command=get_certificate_command, interval=polling_interval
136136
)
137-
return LROPoller(command, create_certificate_operation, None, create_certificate_polling)
137+
return LROPoller(command, create_certificate_operation, lambda *_: None, create_certificate_polling)
138138

139139
@distributed_trace
140140
def get_certificate(self, certificate_name, **kwargs):
@@ -668,13 +668,15 @@ def set_contacts(self, contacts, **kwargs):
668668
:caption: Create contacts
669669
:dedent: 8
670670
"""
671-
contacts = self._client.set_certificate_contacts(
671+
new_contacts = self._client.set_certificate_contacts(
672672
vault_base_url=self.vault_url,
673673
contacts=self._models.Contacts(contact_list=[c._to_certificate_contacts_item() for c in contacts]),
674674
error_map=_error_map,
675675
**kwargs
676676
)
677-
return [CertificateContact._from_certificate_contacts_item(contact_item=item) for item in contacts.contact_list]
677+
return [
678+
CertificateContact._from_certificate_contacts_item(contact_item=item) for item in new_contacts.contact_list
679+
]
678680

679681
@distributed_trace
680682
def get_contacts(self, **kwargs):
@@ -885,7 +887,7 @@ def create_issuer(self, issuer_name, provider, **kwargs):
885887
phone=contact.phone,
886888
)
887889
for contact in admin_contacts
888-
]
890+
] # type: Optional[List[Any]]
889891
else:
890892
admin_details = None
891893
if organization_id or admin_details:
@@ -946,7 +948,7 @@ def update_issuer(self, issuer_name, **kwargs):
946948
phone=contact.phone,
947949
)
948950
for contact in admin_contacts
949-
]
951+
] # type: Optional[List[Any]]
950952
else:
951953
admin_details = None
952954
if organization_id or admin_details:

0 commit comments

Comments
 (0)