Skip to content

build: Add godoclint linter.#3682

Merged
davecgh merged 2 commits intodecred:masterfrom
jholdstock:comments
Apr 24, 2026
Merged

build: Add godoclint linter.#3682
davecgh merged 2 commits intodecred:masterfrom
jholdstock:comments

Conversation

@jholdstock
Copy link
Copy Markdown
Member

The enabled linters have been selected such that they do not force missing comments to be added because this would be far too laborious. They are enough however to ensure the correctness of any comments which are already present.

Also update linter to latest version.

Ensure the godoc comment for all exported (and some private) funcs and
vars have the correct names.
The enabled linters have been selected such that they do not force
missing comments to be added because this would be far too laborious.
They are enough however to ensure the correctness of any comments which
are already present.

Also update linter to latest version.
Copy link
Copy Markdown

@ukane-philemon ukane-philemon left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread .github/workflows/go.yml
with:
install-mode: "goinstall"
version: 72798d34b1eb36745915b0d4eea5c2b3b31bdfe3 # v2.9.0
version: 8f3b0c7ed018e57905fbd873c697e0b1ede605a5 # v2.11.4
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Confirmed 8f3b0c7ed018e57905fbd873c697e0b1ede605a5.

@davecgh davecgh changed the title ci: Add godoclint linter. build: Add godoclint linter. Apr 24, 2026
@davecgh davecgh merged commit 6755842 into decred:master Apr 24, 2026
32 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.

3 participants