Skip to content

Modernize for loop using range over int in SetParamValues#2823

Closed
vishr wants to merge 1 commit intomasterfrom
modernize-for-loop-range-over-int
Closed

Modernize for loop using range over int in SetParamValues#2823
vishr wants to merge 1 commit intomasterfrom
modernize-for-loop-range-over-int

Conversation

@vishr
Copy link
Copy Markdown
Member

@vishr vishr commented Sep 16, 2025

Summary

Modernizes a for loop in context.go to use Go 1.22's new range over int syntax for cleaner iteration.

Changes:

  • Replace for i := 0; i < limit; i++ with for i := range limit in SetParamValues method

Benefits:

  • Cleaner, more idiomatic Go 1.22+ code
  • Slight performance improvement
  • Reduced cognitive load

Test plan

  • All existing tests pass
  • Linting passes
  • No behavioral changes

🤖 Generated with Claude Code

Uses Go 1.22+ range over int syntax for cleaner iteration.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@aldas aldas closed this Jan 20, 2026
@aldas aldas deleted the modernize-for-loop-range-over-int branch February 22, 2026 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants