Skip to content

[Key Vault] Pull keyword-only arguments out of kwargs #31106

Open
@mccoyp

Description

We should update method signatures to use keyword-only argument notation, rather than accepting everything in **kwargs. For example,

def list_properties_of_certificates(self, **kwargs) -> ItemPaged[CertificateProperties]:

should become

def list_properties_of_certificates(self, *, include_pending: Optional[bool] = None, **kwargs) -> ItemPaged[CertificateProperties]:

since include_pending is a recognized keyword-only argument.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

Labels

ClientThis issue points to a problem in the data-plane of the library.KeyVaultMQThis issue is part of a "milestone of quality" initiative.

Type

No type

Projects

  • Status

    Not Started

Relationships

None yet

Development

No branches or pull requests

Issue actions