Skip to content

Add missing resource strings for Get-WinEvent#27397

Merged
daxian-dbw merged 3 commits into
PowerShell:masterfrom
MartinGC94:resourceStrings
May 6, 2026
Merged

Add missing resource strings for Get-WinEvent#27397
daxian-dbw merged 3 commits into
PowerShell:masterfrom
MartinGC94:resourceStrings

Conversation

@MartinGC94
Copy link
Copy Markdown
Contributor

PR Summary

The parameters for Get-WinEvent refer to help strings that don't actually exist.
This PR fixes that by adding the missing strings.
The text was copy+pasted from the existing help text (with some minor editing to cut out unnecessary details).

PR Context

Fixes: #27393

PR Checklist

Copilot AI review requested due to automatic review settings May 4, 2026 19:51
@MartinGC94 MartinGC94 requested a review from a team as a code owner May 4, 2026 19:51
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds the missing GetEventResources entries referenced by Get-WinEvent parameter attributes so parameter-help lookup and tab completion can resolve those strings instead of hitting the missing-resource assertion described in issue #27393.

Changes:

  • Added new resource strings for Get-WinEvent parameter help text in GetEventResources.resx.
  • Covered the missing help IDs for log, provider, path, max events, and computer name parameters.
  • Centralized the fix in the diagnostics resource file used by the cmdlet metadata.

Comment thread src/Microsoft.PowerShell.Commands.Diagnostics/resources/GetEventResources.resx Outdated
Comment thread src/Microsoft.PowerShell.Commands.Diagnostics/resources/GetEventResources.resx Outdated
Comment thread src/Microsoft.PowerShell.Commands.Diagnostics/resources/GetEventResources.resx Outdated
Comment on lines +288 to +289
<data name="ListLogParamHelp" xml:space="preserve">
<value>Specifies the event logs. Wildcards are permitted.</value>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Member

@daxian-dbw daxian-dbw left a comment

Choose a reason for hiding this comment

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

@MartinGC94 Could you please fixes those 2 typos in the resource strings pointed out by Copilot? I will merge after that.

@daxian-dbw daxian-dbw added the CL-General Indicates that a PR should be marked as a general cmdlet change in the Change Log label May 5, 2026
Copy link
Copy Markdown
Member

@daxian-dbw daxian-dbw left a comment

Choose a reason for hiding this comment

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

LGTM!

@daxian-dbw daxian-dbw merged commit 92e80a3 into PowerShell:master May 6, 2026
36 checks passed
JustinGrote pushed a commit to JustinGrote/PowerShell that referenced this pull request Jun 2, 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.

Assertion failed when looking up parameter help message

3 participants