-
Notifications
You must be signed in to change notification settings - Fork 191
Adding -Force switch to ConfirmImpact=High functions #226
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
Adding -Force switch to ConfirmImpact=High functions #226
Conversation
@X-Guardian I've applied your changes 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much for doing this!
Minor formatting change request that differs from the source article, and then this should be ready for CI verification and then merging.
Thanks @X-Guardian for giving this an early review! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent. Thanks so much for doing this. This is definitely going to make the Remove-*
methods easier (and more natural) to use.
Will kick off a CI for this once the current one ends and merge it pending success.
Thanks again for your contribution! Looking forward to see what you pick up next!
#230 would be an easy one to knock off if you're interested in picking up another bug.
/azp run PowerShellForGitHub-CI |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run PowerShellForGitHub-CI |
Azure Pipelines successfully started running 1 pipeline(s). |
Description
Adding -Force switch to ConfirmImpact=High functions and tests
Issues Fixed
Fixes #218
Checklist