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

Write better error message if config file is broken#13496

Merged
adityapatwardhan merged 3 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
iSazonov:add-message-settingsfileiSazonov/PowerShell:add-message-settingsfileCopy head branch name to clipboard
Nov 14, 2020
Merged

Write better error message if config file is broken#13496
adityapatwardhan merged 3 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
iSazonov:add-message-settingsfileiSazonov/PowerShell:add-message-settingsfileCopy head branch name to clipboard

Conversation

@iSazonov

@iSazonov iSazonov commented Aug 21, 2020

Copy link
Copy Markdown
Collaborator

PR Summary

Fix #11964

Write better error message if config file is broken.

PR Context

PR Checklist

@iSazonov iSazonov added the CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log label Aug 21, 2020
@iSazonov iSazonov requested a review from daxian-dbw August 21, 2020 07:57
Comment thread src/System.Management.Automation/resources/PSConfigurationStrings.resx Outdated
@adityapatwardhan

Copy link
Copy Markdown
Member

@TravisEz13 - please update review if the error message is satisfactory

@ghost ghost added the Review - Needed The PR is being reviewed label Sep 2, 2020
@ghost

ghost commented Sep 2, 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 force-pushed the add-message-settingsfile branch from 2d94708 to 54d3ae2 Compare September 2, 2020 03:09
@adityapatwardhan adityapatwardhan merged commit d9090b4 into PowerShell:master Nov 14, 2020
@ghost ghost removed the Review - Needed The PR is being reviewed label Nov 14, 2020
@adityapatwardhan adityapatwardhan added this to the 7.2.0-preview.2 milestone Nov 14, 2020
@iSazonov iSazonov deleted the add-message-settingsfile branch November 14, 2020 07:07
This was referenced Nov 15, 2020
iSazonov pushed a commit that referenced this pull request Nov 15, 2020
@ghost

ghost commented Dec 15, 2020

Copy link
Copy Markdown

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

Handy links:

@ghost ghost mentioned this pull request Dec 15, 2020
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.

Pwsh crash on JsonReaderException

3 participants

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