Skip to content

chore: omit unnecessary reassignment#13438

Merged
rvagg merged 1 commit into
filecoin-project:masterfrom
bigmoonbit:master
Dec 1, 2025
Merged

chore: omit unnecessary reassignment#13438
rvagg merged 1 commit into
filecoin-project:masterfrom
bigmoonbit:master

Conversation

@bigmoonbit
Copy link
Copy Markdown
Contributor

@bigmoonbit bigmoonbit commented Nov 27, 2025

Related Issues

Proposed Changes

The new version of Go has been optimized, and variables do not need to be reassigned.

For more info: https://tip.golang.org/wiki/LoopvarExperiment#does-this-mean-i-dont-have-to-write-x--x-in-my-loops-anymore

Additional Info

Checklist

Before you mark the PR ready for review, please make sure that:

Signed-off-by: bigmoonbit <bigmoonbit@outlook.com>
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz Nov 27, 2025
Copy link
Copy Markdown
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

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

nice, thanks

@github-project-automation github-project-automation Bot moved this from 📌 Triage to ✔️ Approved by reviewer in FilOz Dec 1, 2025
@rvagg rvagg added the skip/changelog This change does not require CHANGELOG.md update label Dec 1, 2025
@rvagg rvagg merged commit 679e71f into filecoin-project:master Dec 1, 2025
97 of 101 checks passed
@github-project-automation github-project-automation Bot moved this from ✔️ Approved by reviewer to 🎉 Done in FilOz Dec 1, 2025
aceppaluni pushed a commit to aceppaluni/lotus that referenced this pull request Dec 2, 2025
Signed-off-by: bigmoonbit <bigmoonbit@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does not require CHANGELOG.md update

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

2 participants