Skip to content

chore: gofmt formatting#872

Merged
mbevc1 merged 1 commit into
mainfrom
20260508_gofmt
May 8, 2026
Merged

chore: gofmt formatting#872
mbevc1 merged 1 commit into
mainfrom
20260508_gofmt

Conversation

@mbevc1
Copy link
Copy Markdown
Contributor

@mbevc1 mbevc1 commented May 8, 2026

Minor formatting fix

@mbevc1 mbevc1 enabled auto-merge (squash) May 8, 2026 15:31
@mbevc1 mbevc1 added tech-debt go Pull requests that update go code labels May 8, 2026
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented May 8, 2026

Claude finished @mbevc1's task in 1m 31s —— View job


Review of PR #872 — gofmt formatting

  • Run static analysis (go build, go vet, go mod tidy)
  • Review the diff for correctness and consistency
  • Post review summary

Static Analysis

All checks pass:

  • go build ./... — clean
  • go vet ./... — clean
  • go mod tidy — no changes to go.mod / go.sum

Review

LGTM. This is a straightforward gofmt alignment fix in cmd/kosli/docs_test.go.

The first cmdTestCase literal (lines 29–33) had its fields (name, cmd, golden) over-padded with extra spaces to match the wider wantError field used in the sibling test cases. gofmt aligns fields independently per composite literal, so this padding was incorrect. The fix brings the alignment in line with what gofmt produces.

No functional change, no risk. Good housekeeping.

@mbevc1 mbevc1 merged commit 0655b5a into main May 8, 2026
20 checks passed
@mbevc1 mbevc1 deleted the 20260508_gofmt branch May 8, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update go code tech-debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants