Skip to content

chore: fix comment formatting and typos across codebase#13607

Open
criciss wants to merge 2 commits into
filecoin-project:masterfrom
criciss:master
Open

chore: fix comment formatting and typos across codebase#13607
criciss wants to merge 2 commits into
filecoin-project:masterfrom
criciss:master

Conversation

@criciss
Copy link
Copy Markdown

@criciss criciss commented May 3, 2026

Related Issues

Proposed Changes

This commit standardizes comment formatting by ensuring consistent spacing
after comment markers (//) and fixes minor typos in comments throughout the
codebase. Changes include:

  • Fixed spacing in inline comments to follow Go style conventions
  • Corrected typo "rance" to "chance" in splitstore_compact.go
  • Removed trailing "0" in funding_stage.go comment
  • Removed extra spaces in multi-line comment formatting
  • Improved readability of comments in storage sealer components

No functional changes were made; this is purely a documentation and
formatting improvement.

Additional Info

Checklist

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

Signed-off-by: criciss <cricis@msn.com>
@github-project-automation github-project-automation Bot moved this to 📌 Triage in FilOz May 3, 2026
Signed-off-by: criciss <cricis@msn.com>
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.

thanks

@github-project-automation github-project-automation Bot moved this from 📌 Triage to ✔️ Approved by reviewer in FilOz May 4, 2026
@rvagg rvagg added the skip/changelog This change does not require CHANGELOG.md update label May 4, 2026
@rvagg
Copy link
Copy Markdown
Member

rvagg commented May 4, 2026

sorry @criciss it's failing the gen check, would you mind running make gen locally and pushing the change? It's most likely the node/config/types.go change that's not being picked up (that also could be fixed manaully if you have any trouble with make gen, you just have to find the instances where it's used)

@rjan90
Copy link
Copy Markdown
Contributor

rjan90 commented May 13, 2026

@criciss are you able to run the make gen on this?

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: ✔️ Approved by reviewer

Development

Successfully merging this pull request may close these issues.

3 participants