Skip to content

[engsys] Remove unused @types/inquirer dependency from api-management-custom-widgets-tools#22929

Merged
timovv merged 1 commit intoAzure:mainfrom
timovv:engsys/22842-remove-inquirer-typedefs
Aug 17, 2022
Merged

[engsys] Remove unused @types/inquirer dependency from api-management-custom-widgets-tools#22929
timovv merged 1 commit intoAzure:mainfrom
timovv:engsys/22842-remove-inquirer-typedefs

Conversation

@timovv
Copy link
Member

@timovv timovv commented Aug 17, 2022

Packages impacted by this PR

  • @azure/api-management-custom-widgets-tools

Issues associated with this PR

Describe the problem that is addressed by this PR

Removes type definition dependency that is unused (there's no corresponding dependency on the actual inquirer package and no references to the types in the code.

@timovv timovv requested a review from xirzec as a code owner August 17, 2022 22:08
@azure-sdk
Copy link
Collaborator

API change check

API changes are not detected in this pull request.

Copy link
Member

@xirzec xirzec left a comment

Choose a reason for hiding this comment

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

Thanks for the cleanup! Feels like we might have an opportunity to write some automation to identify unused deps...

@timovv timovv merged commit f9a971a into Azure:main Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dependency package @types/inquirer has a new version available

3 participants