Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add ErrorAction Stop to DeviceGuard function#2825

Merged
lzybkr merged 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
adityapatwardhan:DeviceGuardFailureadityapatwardhan/PowerShell:DeviceGuardFailureCopy head branch name to clipboard
Dec 2, 2016
Merged

Add ErrorAction Stop to DeviceGuard function#2825
lzybkr merged 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
adityapatwardhan:DeviceGuardFailureadityapatwardhan/PowerShell:DeviceGuardFailureCopy head branch name to clipboard

Conversation

@adityapatwardhan

Copy link
Copy Markdown
Member

A non-terminating error is thrown when the namespace does not exist by
Get-CimInstance. The namespace is not present on OS versions below Windows
10. It is not caught in the try-catch unless ErrorAction Stop is added.

Also fixed a typo in test name.

A non-terminating error is thrown when the namespace does not exist by
Get-CimInstance. The namespace is not present on OS versions below Windows
10. It is not caught in the try-catch unless ErrorAction Stop is added.
@adityapatwardhan

Copy link
Copy Markdown
Member Author

@mirichmo Can you review, since you added this file initially?

@adityapatwardhan adityapatwardhan changed the title Add ErrorAction Stop so DeviceGuard function Add ErrorAction Stop to DeviceGuard function Dec 1, 2016
@PowerShellTeam PowerShellTeam added the Review - Needed The PR is being reviewed label Dec 2, 2016
@mirichmo

mirichmo commented Dec 2, 2016

Copy link
Copy Markdown
Member

I'll merge it once Travis succeeds

@mirichmo mirichmo self-assigned this Dec 2, 2016
@adityapatwardhan

Copy link
Copy Markdown
Member Author

@mirichmo All checks passed.

@lzybkr lzybkr merged commit d9d97a4 into PowerShell:master Dec 2, 2016
@adityapatwardhan adityapatwardhan deleted the DeviceGuardFailure branch December 2, 2016 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review - Needed The PR is being reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.