Fix check for Microsoft Azure CLI substring

OfficeDev / Office-Addin-Scripts

A set of scripts and packages that are consumed in Office add-ins projects.

MIT License

153 stars 93 forks source link

Fix check for Microsoft Azure CLI substring #821

Closed davidchesnut closed 7 months ago

davidchesnut commented 7 months ago

Microsoft Azure CLI (32-bit) and similar strings will no longer match the check. Modified to check for the substring instead.

This is a patch. Needs someone to double-check testing that this works correctly when Azure CLI is installed, and when it is uninstalled.

millerds commented 7 months ago

/azurepipelines run

azure-pipelines[bot] commented 7 months ago

Azure Pipelines successfully started running 1 pipeline(s).

millerds commented 7 months ago

Looks like linux is failing because it's already using node v20 and the dependency we have on teams package (and it's nested dependencies) creates a warning on install which fails the tests. This change is specific to Win32 (an needed) so I'm going to merge it any way.