Skip to content

refactor: replace append slice manipulation with slices.Delete#1992

Merged
chasefleming merged 1 commit into
onflow:masterfrom
socialsister:master
Jul 21, 2025
Merged

refactor: replace append slice manipulation with slices.Delete#1992
chasefleming merged 1 commit into
onflow:masterfrom
socialsister:master

Conversation

@socialsister
Copy link
Copy Markdown
Contributor

@socialsister socialsister commented Jul 16, 2025

Closes #???

Description

Optimize code using a more modern writing style. Official support from Go, for more details visit https://pkg.go.dev/golang.org/x/tools/gopls/internal/analysis/modernize.


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: socialsister <seekseat@qq.com>
Copy link
Copy Markdown
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 for updating this!

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/prompt/prompt.go 0.00% 1 Missing ⚠️
internal/util/util.go 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@chasefleming chasefleming merged commit 75c5640 into onflow:master Jul 21, 2025
7 checks passed
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.

4 participants