Skip to content

Add position 0 to Write-Error parameters#13813

Merged
TravisEz13 merged 2 commits intoPowerShell:masterfrom
ThomasNieto:write_error_position_0
Jul 22, 2021
Merged

Add position 0 to Write-Error parameters#13813
TravisEz13 merged 2 commits intoPowerShell:masterfrom
ThomasNieto:write_error_position_0

Conversation

@ThomasNieto
Copy link
Copy Markdown
Contributor

@ThomasNieto ThomasNieto commented Oct 19, 2020

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
Copy link
Copy Markdown
Collaborator

iSazonov commented Oct 19, 2020

@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
Copy link
Copy Markdown

ghost commented Oct 27, 2020

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
Copy link
Copy Markdown

ghost commented Dec 23, 2020

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

iSazonov commented Jan 9, 2021

@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
Copy link
Copy Markdown

ghost commented Jan 16, 2021

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
Copy link
Copy Markdown

ghost commented Aug 23, 2021

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

Handy links:

@ghost
Copy link
Copy Markdown

ghost commented Sep 28, 2021

🎉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