Skip to content

Normalize whitespace and encoding in source and sample files - #1989

Draft
brendandburns with Copilot wants to merge 3 commits into
masterfrom
copilot/fix-whitespace-encoding-issues
Draft

brendandburns with Copilot wants to merge 3 commits into
masterfrom
copilot/fix-whitespace-encoding-issues

Conversation

Copilot AI commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

Several source and sample files had whitespace, final-newline, and UTF-8 encoding inconsistencies. This change normalizes those issues in the requested files.

  • Whitespace: Remove trailing whitespace and correct indentation.
  • File endings and encoding: Add missing final newlines and remove the UTF-8 BOM.

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:30
@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 whitespace and encoding issues in source and sample files Normalize whitespace and encoding in source and sample files Nov 11, 2026
Copilot AI requested a review from brendandburns November 11, 2026 17:31

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.

🟡 Changes recommended

The argument at tests/KubernetesClient.Tests/WatchTests.cs:72 has 23 leading spaces instead of 24, leaving an inconsistency with the repository’s four-space indentation convention.

1 open issue

Review effort: Balanced

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings November 13, 2026 00:47

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 block the formatting and UTF-8 encoding cleanup.

No open issues 🎉

1 resolved since last review

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 whitespace and encoding issues in source and sample files

3 participants