Skip to content

[release/v7.6.6] Fix comments in CmdletizationCoreResources.resx - #27924

Merged
Patrick Meinecke (SeeminglyScience) merged 1 commit into
PowerShell:release/v7.6.6from
SeeminglyScience:backport/release/v7.6.6/27879-4870e903d
Sep 1, 2026
Merged

Patrick Meinecke (SeeminglyScience) merged 1 commit into
PowerShell:release/v7.6.6from
SeeminglyScience:backport/release/v7.6.6/27879-4870e903d

Conversation

@SeeminglyScience

Copy link
Copy Markdown
Contributor

Backport of #27879 to release/v7.6.6

Triggered by Patrick Meinecke (@SeeminglyScience) on behalf of Dongbo Wang (@daxian-dbw)

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Fixes localization dev-rule comments in CmdletizationCoreResources.resx that referenced string values missing from the resource, required for correct localization pipeline validation.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Cherry-picked cleanly with no conflicts. Comment-only fix, no functional testing required beyond standard CI.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Fixes developer-facing comments/dev-rules in a resx file; no impact on shipped resource values or runtime behavior.

Copilot AI lite review requested due to automatic review settings September 1, 2026 01:20
@SeeminglyScience Patrick Meinecke (SeeminglyScience) added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Sep 1, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Backports a comment-only fix to the release/v7.6.6 branch to correct localization dev-rule comments in CmdletizationCoreResources.resx, removing {StrContains=...} directives that referenced substrings not present in the corresponding resource values (which can break localization pipeline validation).

Changes:

  • Removed invalid {StrContains="CmdletParameterMetadata"}, {StrContains="PSName"}, and {StrContains="CmdletParameterSet"} dev-rule directives from affected resource comments.
  • Simplified the EnumWriter_InvalidValueName dev-rule directive to only require {StrContains="Name"}, matching the actual resource value content.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@SeeminglyScience
Patrick Meinecke (SeeminglyScience) merged commit a0460df into PowerShell:release/v7.6.6 Sep 1, 2026
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants