Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix passing null parameters to Find-WinGetPackage #2092

Merged
merged 1 commit into from
Apr 25, 2022
Merged

fix passing null parameters to Find-WinGetPackage #2092

merged 1 commit into from
Apr 25, 2022

Conversation

bftblomster
Copy link
Contributor

@bftblomster bftblomster commented Apr 13, 2022


This PR enables all cmdlets in the module and makes Install-WinGetPackage actually work. There may be other stuff wrong with the other cmdlets but I didn't test the other ones thoroughly.

Microsoft Reviewers: Open in CodeFlow

@denelon
Copy link
Contributor

denelon commented Apr 14, 2022

It looks like I didn't get "--verbose-log" in there. Did I miss something?

@bftblomster
Copy link
Contributor Author

It looks like I didn't get "--verbose-log" in there. Did I miss something?

I don't see that in the original version. Maybe it wasn't added?

@denelon
Copy link
Contributor

denelon commented Apr 14, 2022

That wouldn't surprise me! Would you mind adding it?

@bftblomster
Copy link
Contributor Author

That wouldn't surprise me! Would you mind adding it?

From what I can tell is Invoke-WinGetCommand would need to have verbose logging support and I think that's outside my abilities to add.

@denelon
Copy link
Contributor

denelon commented Apr 15, 2022

No worries then 😊. I'll try to remember to add it after this PR is resolved.

@bftblomster

This comment was marked as off-topic.

@denelon denelon merged commit 2937bdf into microsoft:master Apr 25, 2022
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.

2 participants