Skip to content

refactor: replace foo += 1 with foo++#299

Merged
tenthirtyam merged 1 commit intomainfrom
fix/revive-increment-decrement-linting
Apr 30, 2025
Merged

refactor: replace foo += 1 with foo++#299
tenthirtyam merged 1 commit intomainfrom
fix/revive-increment-decrement-linting

Conversation

@tenthirtyam
Copy link
Copy Markdown
Collaborator

Description

Replaces foo += 1 with foo++.

This change follows Go's idiomatic style by using the ++ operator instead of += 1 for incrementing variables. The ++ operator is more concise and is the standard way to increment loop counters in Go.

@tenthirtyam tenthirtyam added this to the v1.2.0 milestone Apr 30, 2025
@tenthirtyam tenthirtyam self-assigned this Apr 30, 2025
@tenthirtyam tenthirtyam requested a review from a team as a code owner April 30, 2025 03:03
@tenthirtyam tenthirtyam changed the title refactor: fix/revive-increment-decrement-linting refactor: replace foo += 1 with foo++ Apr 30, 2025
Replaces `foo += 1` with `foo++`.

This change follows Go's idiomatic style by using the `++` operator instead of `+= 1` for incrementing variables. The `++` operator is more concise and is the standard way to increment loop counters in Go.

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam tenthirtyam force-pushed the fix/revive-increment-decrement-linting branch from 3af2ff9 to 24108b3 Compare April 30, 2025 03:06
@tenthirtyam tenthirtyam merged commit 507b846 into main Apr 30, 2025
14 checks passed
@tenthirtyam tenthirtyam deleted the fix/revive-increment-decrement-linting branch April 30, 2025 16:01
@github-actions
Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants