Skip to content
This repository was archived by the owner on Sep 10, 2026. It is now read-only.

Drop every em dash - #28

Merged
seenu-k merged 1 commit into
mainfrom
docs/no-em-dashes
Aug 8, 2026
Merged

seenu-k merged 1 commit into
mainfrom
docs/no-em-dashes

Conversation

@seenu-k

@seenu-k seenu-k commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

367 of them, across 98 tracked files. Recast per sentence as a comma, colon, semicolon, parenthesis or a second sentence, not swept with a regex. No spaced hyphens were introduced.

Everything, not just the public API docs: lib/, bin/, the example game, the tests, MAINTAINERS.md / CONTRIBUTING.md / AGENTS.md / README.md, fonts/README.md, both pubspecs, build.yaml, .gitignore, the workflows and the changelog.

Text a player actually reads

Was Now
The game updated — try again. The game updated. Try again.
Blocked — enable notifications in this site's browser settings Blocked: enable notifications …
Disabled — tap to open Settings Disabled: tap to open Settings
Account created — your games, ratings, and friends are saved. Account created. Your games, ratings, and friends are saved.
Game in progress — the replay will be available when it … Game in progress. The replay will be available when it …
configure_firebase printed a bare — for an unconfigured platform prints none

test/core/errors/error_messages_test.dart asserted the first of those and moves with it.

Both pubspec.yaml descriptions change wording, so both pub.dev listings do too.

Generated code

The four *.g.dart files carry riverpod_generator's copy of the doc comment above each provider, so dart run build_runner build --delete-conflicting-outputs brings them along. Committed, and the drift guard in checks.yml covers it.

Changelog

One cider log changed entry, per CONTRIBUTING. The heading link-reference guard is untouched — only prose inside existing entries changed.

Verification

flutter analyze clean · dart format clean · 215 package tests and 21 example tests pass.

Companion PRs: eigeninteractive/eigen-server#46 and eigeninteractive/eigen-web#32. Between the three, zero em dashes remain in any tracked file in any of the repositories.

🤖 Generated with Claude Code

367 of them across 98 tracked files, which is enough that the prose reads
as machine-written whatever it actually says. Recast per sentence as a
comma, colon, semicolon, parenthesis or a second sentence, not swept with
a regex. No spaced hyphens were introduced.

Everything, not just the public API docs: lib/, bin/, the example game,
the tests, MAINTAINERS/CONTRIBUTING/AGENTS/README, fonts/README.md, both
pubspecs, build.yaml, .gitignore, the workflows and the changelog.

Text a player actually reads changed with it:

  · `The game updated. Try again.` for a stateUpdated rejection.
  · The notification-permission rows in settings: `Blocked: enable
    notifications…` and `Disabled: tap to open Settings`.
  · `Account created. Your games, ratings, and friends are saved.`
  · `Game in progress. The replay will be available when it …`
  · `configure_firebase`'s summary prints `none` where a platform was
    not configured, rather than a bare dash.

pubspec descriptions move too, so both pub.dev listings change wording.

The four *.g.dart files carry riverpod_generator's copy of the doc
comments above each provider; `build_runner build` brings them along.

analyze clean, format clean, 215 package tests and 21 example tests pass.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@seenu-k
seenu-k merged commit 220b204 into main Aug 8, 2026
3 checks passed
@seenu-k
seenu-k deleted the docs/no-em-dashes branch August 8, 2026 11:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant