-
Notifications
You must be signed in to change notification settings - Fork 7.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make Update-Help
throw proper error when current culture is not associated with a language
#19765
Make Update-Help
throw proper error when current culture is not associated with a language
#19765
Conversation
@microsoft-github-policy-service agree |
1489fe0
to
b159db3
Compare
src/System.Management.Automation/resources/HelpDisplayStrings.resx
Outdated
Show resolved
Hide resolved
src/System.Management.Automation/resources/HelpDisplayStrings.resx
Outdated
Show resolved
Hide resolved
@adityapatwardhan : ok, so that's an option. See how it looks like. |
@josea I like the one above. Please push the changes and I will re-review. Thanks! |
Also, please check CI failures for the test you added. |
034c974
to
b608697
Compare
b608697
to
7c3f5b6
Compare
@adityapatwardhan : changes are done. CI now passes. |
@josea CIs are all failing, please take a look. |
7c3f5b6
to
014217b
Compare
The PR has unrelated commits, it needs to be rebased. |
src/System.Management.Automation/resources/HelpDisplayStrings.resx
Outdated
Show resolved
Hide resolved
….UTF-8 culture (issue 19765).
014217b
to
df3a79e
Compare
@adityapatwardhan : I cleaned the commit. @daxian-dbw , @adityapatwardhan : I am having issues with the tests. They are failing and I am not sure it is related to my changes. See image below - it seems a test is setting up a web listener and it times out. I have no way to re-run the test (other than change the commit forcing it to happen again, but ideally I would just re-run this test to see what is happening). Still, the error doesn't seem to be related to my changes. Please advise. |
@josea pushed 1 small change. Rest looks good. The test failure is intermittent and does typically work fine on re-run. |
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
Update-Help
throw proper error when current culture is not associated with a language
🎉 Handy links: |
PR Summary
Fix #19633
Fix logic to check for no language in current culture (linux C.UTF-8).
PR Context
Check the current culture (if no parameter is passed). If no language is found, throw exception with the designated message.
PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files have the correct copyright headerWIP:
or[ WIP ]
to the beginning of the title (theWIP
bot will keep its status check atPending
while the prefix is present) and remove the prefix when the PR is ready.(which runs in a different PS Host).