Skip to content

ASH-220: gofmt the repo + add a gofmt CI gate#13

Merged
cstaszak merged 1 commit into
mainfrom
cstaszak/ash-220-gofmt
May 22, 2026
Merged

ASH-220: gofmt the repo + add a gofmt CI gate#13
cstaszak merged 1 commit into
mainfrom
cstaszak/ash-220-gofmt

Conversation

@cstaszak

Copy link
Copy Markdown
Contributor

The repo had accumulated gofmt drift across 71 files — struct-field and map-key alignment, and Go 1.19+ doc-comment list/bullet reformatting. ci.yml had no gofmt step, so it went unnoticed (surfaced while wrapping the post-public-release review).

  • gofmt -w . — pure formatting, no behavior change. Full test suite (54 packages) passes unchanged; go vet clean.
  • ci.yml: a gofmt gate after vet (test -z "$(gofmt -l .)") so drift cannot re-accumulate.

Large diff, but it is entirely mechanical formatting — kept as its own PR so it doesn't bury substantive changes (per the plan in ASH-220).

🤖 Generated with Claude Code

The repo had accumulated gofmt drift across 71 files — struct-field and
map-key alignment, and Go 1.19+ doc-comment list/bullet reformatting.
ci.yml had no gofmt step, so it went unnoticed.

- gofmt -w . — pure formatting, no behavior change. Full test suite
  (54 packages) passes unchanged.
- ci.yml: add a gofmt gate (`test -z "$(gofmt -l .)"`) after `vet`, so
  formatting drift cannot re-accumulate.
- Regenerate docs/vocab/inventory.json — it records source file:line
  locations, which shifted when gofmt re-flowed blank lines.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@cstaszak
cstaszak force-pushed the cstaszak/ash-220-gofmt branch from b04e655 to 9d5884d Compare May 22, 2026 00:36
@cstaszak
cstaszak merged commit 13a7823 into main May 22, 2026
5 checks passed
@cstaszak
cstaszak deleted the cstaszak/ash-220-gofmt branch May 22, 2026 00:37
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