Skip to content

C#: Rename increment/decrement helper to PascalCase in SymbolExtensions#21843

Closed
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-code-review-suggestion
Closed

C#: Rename increment/decrement helper to PascalCase in SymbolExtensions#21843
Copilot wants to merge 1 commit into
mainfrom
copilot/fix-code-review-suggestion

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 13, 2026 •
edited
Loading

Bug Fix

What was the bug?

This PR addresses a single review comment on PR #21827. A private helper method in SymbolExtensions used lower camel case (isIncrementOrDecrement), which was inconsistent with surrounding C# naming conventions.

How did you fix it?

  • Naming consistency
    • Renamed isIncrementOrDecrement to IsIncrementOrDecrement.
    • Updated its only call site in TryGetOperatorSymbol.
private static bool IsIncrementOrDecrement(string operatorName) =>
    operatorName == "++" || operatorName == "--";

Testing

  • Not part of the requested change scope; this PR is intentionally limited to the referenced review comment only.

Copilot AI changed the title [WIP] Fix code according to review comment C#: Rename increment/decrement helper to PascalCase in SymbolExtensions May 13, 2026
Copilot AI requested a review from michaelnebel May 13, 2026 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@michaelnebel michaelnebel Awaiting requested review from michaelnebel

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

@michaelnebel michaelnebel

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@michaelnebel