Skip to content

Resolve active SA1116 and SA1118 warnings - #1987

Draft
brendandburns with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-active-stylecop-parameter-layout-warnings
Draft

brendandburns with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-active-stylecop-parameter-layout-warnings

Conversation

Copilot AI commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Clear the enabled StyleCop parameter-layout findings in the reported production and test files while preserving behavior.

  • Production: Reformat the WebSocket constructor arguments and keep the comparison argument on one line.
  • Tests: Reflow the YAML assertion and bind the leader-election failure message before the assertion.
var muxer = new StreamDemuxer(
    webSocket,
    supportsClose: webSocket.SubProtocol == WebSocketProtocol.V5BinaryWebsocketProtocol);

Copilot AI balanced review requested due to automatic review settings November 11, 2026 17:29
@kubernetes-prow kubernetes-prow Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 11, 2026
@linux-foundation-easycla

linux-foundation-easycla Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

CLA Not Signed

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.

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@kubernetes-prow kubernetes-prow Bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Nov 11, 2026
Copilot AI linked an issue Nov 11, 2026 that may be closed by this pull request
Co-authored-by: brendandburns <5751682+brendandburns@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings November 11, 2026 17:34
@kubernetes-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Copilot
Once this PR has been reviewed and has the lgtm label, please ask for approval from brendandburns. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubernetes-prow kubernetes-prow Bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 11, 2026
Copilot AI changed the title [WIP] Fix active StyleCop parameter-layout warnings Resolve active SA1116 and SA1118 warnings Nov 11, 2026
Copilot AI requested a review from brendandburns November 11, 2026 17:35

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.

🟢 Approval recommended

No active review issues are reported for the parameter-layout formatting changes.

No open issues 🎉

Review effort: Balanced

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix active StyleCop parameter-layout warnings

3 participants