Skip to content

Tidy blink: dead assignments, stdlib sort, Kind name array, doc comments - #190

Merged
TomasVotruba merged 1 commit into
mainfrom
blink-cleanup-idioms
Oct 9, 2026
Merged

TomasVotruba merged 1 commit into
mainfrom
blink-cleanup-idioms

Conversation

@TomasVotruba

Copy link
Copy Markdown
Member

Low-risk idiom and lint cleanup from the codebase audit. No behavior change.

  • Remove two dead _ = silencers (no_trailing_whitespace.go, doctrine_annotation_parser.go) and the lastToken variable they were masking, which was assigned but never read.
  • Replace sort.Slice/sort.SliceStable with slices.SortFunc/slices.SortStableFunc in the runner and the registry (drops the sort import).
  • Turn kindNames from a map into an array indexed by the contiguous Kind iota.
  • Add doc comments to the Fixer, Config and Stream types.

…comments

- Remove dead _= silencers and the unused lastToken variable.
- sort.Slice/SliceStable -> slices.SortFunc/SortStableFunc in runner and registry.
- kindNames map -> array indexed by the contiguous Kind iota.
- Add doc comments to the Fixer, Config and Stream types.
@TomasVotruba
TomasVotruba merged commit 925cb93 into main Oct 9, 2026
21 checks passed
@TomasVotruba
TomasVotruba deleted the blink-cleanup-idioms branch October 9, 2026 19:23
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