Skip to content

chore: fix some minor issues in comments#2853

Open
cuoguojida wants to merge 1 commit into
ssvlabs:stagefrom
cuoguojida:stage
Open

chore: fix some minor issues in comments#2853
cuoguojida wants to merge 1 commit into
ssvlabs:stagefrom
cuoguojida:stage

Conversation

@cuoguojida
Copy link
Copy Markdown

fix some minor issues in comments

Signed-off-by: cuoguojida <cuoguojida@outlook.com>
@cuoguojida cuoguojida requested review from a team as code owners May 17, 2026 16:43
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 17, 2026

Greptile Summary

This PR fixes minor grammatical and stylistic issues in code comments across three files — no logic or behaviour changes are included.

  • storage/badger/badger.go: Four Go doc comments updated to the idiomatic third-person form (Get returns …, GetMany returns …, Delete deletes …, CountPrefix returns …).
  • protocol/v2/ssv/runner/runner.go: Duplicate word "validated" removed from a function doc comment.
  • network/p2p/p2p.go: Comma added and verb tense corrected in a function doc comment.

Confidence Score: 5/5

All changes are restricted to code comments; no executable code is modified.

Every diff line touches only comment text. The fixes are correct (idiomatic Go doc style, removed duplicate word, grammar correction), and there is no risk of runtime regression.

No files require special attention.

Important Files Changed

Filename Overview
network/p2p/p2p.go Grammar fix in a comment: added comma and changed "passing" to "passes" for clearer phrasing.
protocol/v2/ssv/runner/runner.go Removed a duplicate "validated" word from a function doc comment.
storage/badger/badger.go Fixed four Go doc comments to use the conventional "Func returns/does …" third-person form (Get, GetMany, Delete, CountPrefix).

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR: fix comments] --> B[network/p2p/p2p.go\ngrammar fix in bootstrapDiscovery comment]
    A --> C[protocol/v2/ssv/runner/runner.go\nremove duplicate 'validated' word]
    A --> D[storage/badger/badger.go\nfix 4 Go doc comments to third-person form]
    B --> E[No logic change]
    C --> E
    D --> E
Loading

Reviews (1): Last reviewed commit: "chore: fix some minor issues in comments" | Re-trigger Greptile

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.

1 participant