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 position 0 to Write-Error parameters#13813

Merged
TravisEz13 merged 2 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
ThomasNieto:write_error_position_0ThomasNieto/PowerShell:write_error_position_0Copy head branch name to clipboard
Jul 22, 2021
Merged

Add position 0 to Write-Error parameters#13813
TravisEz13 merged 2 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
ThomasNieto:write_error_position_0ThomasNieto/PowerShell:write_error_position_0Copy head branch name to clipboard

Conversation

@ThomasNieto

@ThomasNieto ThomasNieto commented Oct 19, 2020

Copy link
Copy Markdown
Contributor

PR Summary

Add Position = 0 to Write-Error to Exception and ErrorRecord parameters.

PR Context

Resolves #10739

PR Checklist

@ghost ghost assigned TravisEz13 Oct 19, 2020
@iSazonov

iSazonov commented Oct 19, 2020

Copy link
Copy Markdown
Collaborator

@ThomasNieto Please open new issue for 7.2 version in PowerShell-Docs repo.

@iSazonov iSazonov added Documentation Needed in this repo Documentation is needed in this repo CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log Hacktoberfest-Accepted Accepted to participate in Hacktoberfest labels Oct 19, 2020
@iSazonov iSazonov removed the Documentation Needed in this repo Documentation is needed in this repo label Oct 20, 2020
@ghost ghost added the Review - Needed The PR is being reviewed label Oct 27, 2020
@ghost

ghost commented Oct 27, 2020

Copy link
Copy Markdown

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

@iSazonov

Copy link
Copy Markdown
Collaborator

@ThomasNieto Please resolve merge conflicts.

@ghost ghost removed the Review - Needed The PR is being reviewed label Dec 16, 2020
@ghost ghost added the Review - Needed The PR is being reviewed label Dec 23, 2020
@ghost

ghost commented Dec 23, 2020

Copy link
Copy Markdown

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

@iSazonov

iSazonov commented Jan 9, 2021

Copy link
Copy Markdown
Collaborator

@ThomasNieto Please resolve merge conflicts.

@ghost ghost removed the Review - Needed The PR is being reviewed label Jan 9, 2021
@ghost ghost added the Review - Needed The PR is being reviewed label Jan 16, 2021
@ghost

ghost commented Jan 16, 2021

Copy link
Copy Markdown

This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days.
Maintainer, please provide feedback and/or mark it as Waiting on Author

@SteveL-MSFT SteveL-MSFT force-pushed the write_error_position_0 branch from 2ab1bbe to 162ffaa Compare July 21, 2021 22:42
@SteveL-MSFT SteveL-MSFT removed the Review - Needed The PR is being reviewed label Jul 21, 2021
@TravisEz13 TravisEz13 merged commit 05943c6 into PowerShell:master Jul 22, 2021
@iSazonov iSazonov added this to the 7.2.0-preview.8 milestone Jul 22, 2021
@ghost

ghost commented Aug 23, 2021

Copy link
Copy Markdown

🎉v7.2.0-preview.9 has been released which incorporates this pull request.:tada:

Handy links:

@ghost

ghost commented Sep 28, 2021

Copy link
Copy Markdown

🎉v7.2.0-preview.10 has been released which incorporates this pull request.:tada:

Handy links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log Hacktoberfest-Accepted Accepted to participate in Hacktoberfest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Write-Error should use positional binding to -ErrorRecord and -Exception

4 participants

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