microsoft / artifacts-credprovider

The Azure Artifacts Credential Provider enables dotnet, NuGet.exe, and MSBuild to interactively acquire credentials for Azure Artifacts feeds.
MIT License
787 stars 705 forks source link

Unnecessary output in PowerShell script #278

Closed czdietrich closed 3 years ago

czdietrich commented 3 years ago

Executing the installcredprovider.ps1 currently writes out unnecessary information when the temp folder for extracting the ZIP file is created.

Fetching release https://api.github.com/repos/Microsoft/artifacts-credprovider/releases/latest
>
>
>     Directory: C:\Users\AzDevOps\AppData\Local\Temp\1
> 
> 
> Mode                 LastWriteTime         Length Name                                                                 
> ----                 -------------         ------ ----                                                                 
> d-----        16.11.2021     15:55                CredProviderZip                                                      
Downloading https://github.com/microsoft/artifacts-credprovider/releases/download/v0.1.28/Microsoft.NetCore2.NuGet.CredentialProvider.zip to C:\Users\AzDevOps\AppData\Local\Temp\1\CredProviderZip\Microsoft.NetCore2.NuGet.CredentialProvider.zip

It would be nice if we could get rid of the output for the folder creation command: New-Item -ItemType Directory -Force -Path $tempZipLocation.

Probably it is enough to simply add > $null to the end of the line.

github-actions[bot] commented 3 years ago

In order to consolidate to fewer feedback channels, we've moved suggestions and issue reporting to Developer Community.