Skip to content

Fix tab completion for unlocalized about* topics - #15265

Merged
Aditya Patwardhan (adityapatwardhan) merged 5 commits into
PowerShell:masterfrom
MartinGC94:FixHelpCompletion
May 5, 2021
Merged

Fix tab completion for unlocalized about* topics#15265
Aditya Patwardhan (adityapatwardhan) merged 5 commits into
PowerShell:masterfrom
MartinGC94:FixHelpCompletion

Conversation

@MartinGC94

Copy link
Copy Markdown
Contributor

Fixes tab completion for help topics on systems without localized help by adding en-US as a fallback.
Get-Help About<Tab>

PR Context

Get-Help will automatically fall back to en-US help topics so the completion for it should do the same.

PR Checklist

Comment thread src/System.Management.Automation/engine/CommandCompletion/CompletionCompleters.cs Outdated
Comment thread src/System.Management.Automation/engine/CommandCompletion/CompletionCompleters.cs Outdated
Comment thread src/System.Management.Automation/engine/CommandCompletion/CompletionCompleters.cs Outdated
Comment on lines +6280 to +6283
Path.Combine(userHelpDir, currentCulture),
Path.Combine(userHelpDir, "en-US"),
Path.Combine(appHelpDir, currentCulture),
Path.Combine(appHelpDir, "en-US")

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For discussion:

Suggested change
Path.Combine(userHelpDir, currentCulture),
Path.Combine(userHelpDir, "en-US"),
Path.Combine(appHelpDir, currentCulture),
Path.Combine(appHelpDir, "en-US")
Path.Combine(userHelpDir, currentCulture),
Path.Combine(appHelpDir, currentCulture),
Path.Combine(userHelpDir, "en-US"),
Path.Combine(appHelpDir, "en-US")

Maybe users want to get topics in their language first?

Comment thread src/System.Management.Automation/engine/CommandCompletion/CompletionCompleters.cs Outdated
Comment thread test/powershell/Host/TabCompletion/TabCompletion.Tests.ps1 Outdated
@ghost ghost added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Apr 20, 2021
@ghost ghost removed the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Apr 20, 2021
@iSazonov Ilya (iSazonov) added the CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log label Apr 21, 2021
@ghost ghost added the Review - Needed The PR is being reviewed label Apr 28, 2021
@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

@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit 5d8336a into PowerShell:master May 5, 2021
@ghost ghost removed the Review - Needed The PR is being reviewed label May 5, 2021
@adityapatwardhan

Copy link
Copy Markdown
Member

MartinGC94 Thank you for your contribution!

@ghost

Copy link
Copy Markdown

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

Handy links:

@MartinGC94
MartinGC94 deleted the FixHelpCompletion branch June 5, 2022 08:28
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.

3 participants