Skip to content

docs: drop '6 words' from README demo stat bar#2

Merged
bladysh merged 1 commit into
mainfrom
docs/drop-words-segment-from-readme-demo
May 17, 2026
Merged

docs: drop '6 words' from README demo stat bar#2
bladysh merged 1 commit into
mainfrom
docs/drop-words-segment-from-readme-demo

Conversation

@bladysh

@bladysh bladysh commented May 17, 2026

Copy link
Copy Markdown
Owner

The demo terminal block in all 6 READMEs still showed the old stat bar format ('56% saved · 31 chars · 6 words · 2 passes · <1ms'), but commit 4f76f20 ('stat bar: drop N words segment') removed the words segment from the actual CLI output. Align the docs with what users will actually see.

No code changes, no version bump.

What does this PR do?

Type of change

  • Bug fix
  • New feature
  • Documentation
  • Dictionary / language pack change
  • Refactor or chore (no behaviour change)

Checklist

  • I tested my changes locally
  • go build ./... passes
  • go test ./... passes (or make test)
  • go vet ./... and gofmt -l . are clean
  • I only changed what this PR describes

Dictionary changes — extra checklist

  • I read pkg/filters/dictionaries/README.md
  • meta.version was bumped in every touched JSON file
  • Folder name equals meta.lang (ISO 639-1, lowercase)
  • No entry appears in both fillers.json and abbreviations.json
  • I added a 1-line justification per entry (in commit body or PR description)

The demo terminal block in all 6 READMEs still showed the old stat
bar format ('56% saved · 31 chars · 6 words · 2 passes · <1ms'),
but commit 4f76f20 ('stat bar: drop N words segment') removed the
words segment from the actual CLI output. Align the docs with what
users will actually see.

No code changes, no version bump.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@bladysh bladysh merged commit 3621d91 into main May 17, 2026
1 check passed
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