Skip to content

Write better error message if config file is broken - #13496

Merged
Aditya Patwardhan (adityapatwardhan) merged 3 commits into
PowerShell:masterfrom
iSazonov:add-message-settingsfile
Nov 14, 2020
Merged

Write better error message if config file is broken#13496
Aditya Patwardhan (adityapatwardhan) merged 3 commits into
PowerShell:masterfrom
iSazonov:add-message-settingsfile

Conversation

@iSazonov

@iSazonov Ilya (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 Ilya (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
Comment thread src/System.Management.Automation/resources/PSConfigurationStrings.resx Outdated
@adityapatwardhan

Copy link
Copy Markdown
Member

Travis Plunk (@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

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
Ilya (iSazonov) force-pushed the add-message-settingsfile branch from 2d94708 to 54d3ae2 Compare September 2, 2020 03:09
@adityapatwardhan
Aditya Patwardhan (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
@iSazonov
Ilya (iSazonov) deleted the add-message-settingsfile branch November 14, 2020 07:07
This was referenced Nov 15, 2020
Ilya (iSazonov) pushed a commit that referenced this pull request Nov 15, 2020
@ghost

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
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
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