Skip to content

Comments

refactor: use the built-in max/min to simplify the code#2279

Merged
chasefleming merged 1 commit intoonflow:masterfrom
moveyield:master
Feb 23, 2026
Merged

refactor: use the built-in max/min to simplify the code#2279
chasefleming merged 1 commit intoonflow:masterfrom
moveyield:master

Conversation

@moveyield
Copy link
Contributor

@moveyield moveyield commented Feb 15, 2026

Closes #???

Description

In Go 1.21, the standard library includes built-in max/min function, which can greatly simplify the code.


For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

Signed-off-by: moveyield <moveyield@outlook.com>
Copy link
Member

@chasefleming chasefleming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@chasefleming chasefleming merged commit 3a948d7 into onflow:master Feb 23, 2026
11 checks passed
@chasefleming chasefleming added the Improvement Technical work without new features, refactoring, improving tests label Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement Technical work without new features, refactoring, improving tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants