Skip to content

Conversation

@HowardWolosky
Copy link
Contributor

Per helpful feedback from @StartAutomating:

Taking a brief look at implementation, I don't see where you send -Completed message to Write-Progress.

This will cause progress bars to stay visible while any other commands in the script execute. E.g. if you ran a Get- that returned many results, and piped to an expensive Foreach-Object, your progress bar wouldn't go away until the Foreach-Object ended."

Per helpful feedback from @StartAutomating:
  "Taking a brief look at implementation, I don't see where you send -Completed message to Write-Progress.
  This will cause progress bars to stay visible while any other commands in the script execute.
  E.g. if you ran a Get- that returned many results, and piped to an expensive Foreach-Object,
  your progress bar wouldn't go away until the Foreach-Object ended."
@HowardWolosky HowardWolosky merged commit bb2ad45 into microsoft:master May 12, 2020
@HowardWolosky HowardWolosky deleted the completeProgress branch May 12, 2020 20:43
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.

1 participant