Skip to content

chore(golangci-lint): enable godot and fix errors#687

Merged
julieqiu merged 3 commits intomainfrom
cl1
Jun 30, 2025
Merged

chore(golangci-lint): enable godot and fix errors#687
julieqiu merged 3 commits intomainfrom
cl1

Conversation

@julieqiu
Copy link
Copy Markdown
Member

godot is a linter that checks if all top-level comments contain a period at the end of the last sentence if needed. Enable it as part of golangci-lint.

For #638

godot is a linter that checks if all top-level comments contain a period
at the end of the last sentence if needed. Enable it as part of
golangci-lint.

For #638
@julieqiu julieqiu requested a review from jskeet June 28, 2025 17:33
@julieqiu julieqiu enabled auto-merge (squash) June 28, 2025 17:33
jba
jba previously approved these changes Jun 29, 2025
@julieqiu julieqiu changed the title chore(all): turn on godot chore(golangci-lint): turn on godot Jun 30, 2025
@julieqiu julieqiu changed the title chore(golangci-lint): turn on godot chore(golangci-lint): enable godot Jun 30, 2025
@julieqiu julieqiu changed the title chore(golangci-lint): enable godot chore(golangci-lint): enable godot and fix errors Jun 30, 2025
Copy link
Copy Markdown
Contributor

@jskeet jskeet left a comment

Choose a reason for hiding this comment

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

Approving as-is, but with one change that we may or may not want to include in this PR.

@julieqiu julieqiu merged commit e2f511b into main Jun 30, 2025
5 checks passed
@julieqiu julieqiu deleted the cl1 branch June 30, 2025 15:56
julieqiu added a commit that referenced this pull request Jul 31, 2025
godot is a linter that checks if all top-level comments contain a period
at the end of the last sentence if needed. Enable it as part of
golangci-lint.

For #638
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