Azure / pcs-cli

A CLI for deploying Azure IoT PCS

MIT License

33 stars 37 forks source link

update inquirer version to fix script haning issue with node 10.13.0 #432

Closed sakimsft closed 5 years ago

sakimsft commented 5 years ago

Description and Motivation

step to reproduce:

remove all node_modules (global and local)
clear cache (npm cache clean --force)
install RM (pcs -t remotemonitoring -s basic -r dotnet)

While provisioning RM, script hang because inquirer never returns from creating a prompt. It was fixed by using the latest inquirer.

Tested both Node.js version 8.12.0 and 10.13.0

Change type

[ ] Bug fix
[x] New feature
[ ] Enhancement
[ ] Breaking change (breaks backward compatibility)

Checklist:

[ ] All tests passed
[ ] The code follows the code style and conventions of this project
[ ] The change requires a change to the documentation
[ ] I have updated the documentation accordingly

This change is