NotificationsYou must be signed in to change notification settings
Fork45
Star349

Fix regression in get-all default output#874

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

SteveL-MSFT merged 1 commit intoPowerShell:mainfromSteveL-MSFT:get-all-regression

Jun 9, 2025

Merged

Fix regression in get-all default output#874

SteveL-MSFT merged 1 commit intoPowerShell:mainfromSteveL-MSFT:get-all-regression

Jun 9, 2025

Conversation

Copy link

Member

SteveL-MSFT commentedJun 9, 2025

PR Summary

In the original change to supportjson-array output, the conversion fromGetOutputFormat toOutputFormat didn't correctly handleNone which ended up in the anything else default to JSON format whereasNone should return YAML when used interactively. The fix is to make sureNone gets propagated correctly.

Also updated theProcess resource, which is only used for testing, to only emit 3 instances rather than having it scroll off the terminal.

Finally, fixed a non-blocking test issue in the WinPS test that would fail on non-Windows during cleanup.