Skip to content

Conversation

@SteveL-MSFT
Copy link
Member

@SteveL-MSFT SteveL-MSFT commented Feb 25, 2020

PR Summary

This is a regression from a prior change that replaced usage of `n with $newline but accidentally replaced a correct usage of `n as the PositionMessage member uses `n as the newline is from a resource string. Fix is to change back usage of Split() with the newline char rather than $newline which may contain CRLF. This is a regression introduced in rc.3 from rc.2

PR Context

The error message was inadvertently including the legacy line position information.

PR Checklist

@ghost ghost assigned iSazonov Feb 25, 2020
@SteveL-MSFT SteveL-MSFT added this to the GA-consider milestone Feb 25, 2020
@iSazonov iSazonov added the CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log label Feb 25, 2020
@iSazonov iSazonov merged commit 2db8516 into PowerShell:master Feb 26, 2020
@ghost
Copy link

ghost commented Mar 4, 2020

🎉v7.0.0 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants