Skip to content

Check gofmt in CI and reformat - #528

Merged
simonchrz merged 1 commit into
mainfrom
gofmt-ci
Oct 19, 2026
Merged

simonchrz merged 1 commit into
mainfrom
gofmt-ci

Conversation

@andrew

@andrew andrew commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

golangci-lint now runs the gofmt formatter, so unformatted files fail the existing lint job. internal/handler/npm_test.go was the only file on main that needed it; the change there is whitespace.

https://claude.ai/code/session_01NxsA75HBjdWQKcdx7FDTDc

Copilot AI balanced review requested due to automatic review settings October 18, 2026 20:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@simonchrz simonchrz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. With the v2 formatters block, the existing lint job now catches unformatted files.

@simonchrz
simonchrz merged commit 2e0f132 into main Oct 19, 2026
10 checks passed
@simonchrz
simonchrz deleted the gofmt-ci branch October 19, 2026 05:47
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