Skip to content

[sort-imports] enforce sort-imports rule in keyvault-admin#20111

Merged
jeremymeng merged 1 commit intoAzure:mainfrom
bzhang0:lint-keyvault-admin
Jan 31, 2022
Merged

[sort-imports] enforce sort-imports rule in keyvault-admin#20111
jeremymeng merged 1 commit intoAzure:mainfrom
bzhang0:lint-keyvault-admin

Conversation

@bzhang0
Copy link
Copy Markdown
Contributor

@bzhang0 bzhang0 commented Jan 28, 2022

This PR reinforces the changes made in #19128 by changing the subdirectory's linting rule to "sort-imports": "error".

This affects the directory under sdk/keyvault/keyvault-admin.

@ghost ghost added KeyVault customer-reported Issues that are reported by GitHub users external to the Azure organization. labels Jan 28, 2022
@ghost
Copy link
Copy Markdown

ghost commented Jan 28, 2022

Thank you for your contribution bzhang0! We will review the pull request and get back to you soon.

@ghost ghost added the Impact++ This pull request was submitted by a member of the Impact++ team. label Jan 28, 2022
Copy link
Copy Markdown
Member

@jeremymeng jeremymeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@jeremymeng jeremymeng merged commit 3f3f871 into Azure:main Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

customer-reported Issues that are reported by GitHub users external to the Azure organization. Impact++ This pull request was submitted by a member of the Impact++ team. KeyVault

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants