Skip to content

Conversation

@SteveL-MSFT
Copy link
Member

@SteveL-MSFT SteveL-MSFT commented Dec 18, 2019

PR Summary

Some terminals can turn the script:line into a hyperlink, so adding the line number right after the filename allows it to be clickable and takes you to the line of the error instead of just the file. This does make the line number information redundant, but is not a big issue. Since this is similar to what users already saw in NormalView, they should understand what that number represents.

However, if we are running in vscode, it also understands script:line:col making the link click to the location of the error so we add column information only for vscode terminal.

Get-ChildItem: /Users/steve/test/multierror.ps1:3
Line |
   3 | Get-Childitem -Path .\nothere
     | ^ Cannot find path '/Users/steve/repos/PowerShell/nothere' because it does not exist.

PR Context

Fix #10877

PR Checklist

@SteveL-MSFT SteveL-MSFT added this to the GA-consider milestone Dec 18, 2019
@iSazonov iSazonov added the CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log label Dec 18, 2019
@anmenaga anmenaga merged commit 563b1cb into PowerShell:master Dec 18, 2019
@daxian-dbw daxian-dbw modified the milestones: GA-approved, 7.0.0-rc.2 Jan 11, 2020
@ghost
Copy link

ghost commented Jan 16, 2020

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

Some concerns about error formatting with ErrorView = ConciseView

6 participants