Remove em dashes from code and docs

Breadway commented

2026-09-09 08:39:33 +08:00

Owner

Replaces every em dash across Rust source, doc comments, Markdown docs, the vendored openscreen PATCHES.md notes, and CI workflow YAML with colons, semicolons, commas, or parentheses, chosen per-instance for what reads most naturally. No intentional em-dash placeholder glyphs exist in this repo, so this is a full, exception-free cleanup. Also carries along a small pending AGENTS.md "Persistent memory" note that was already staged on main before this branch was cut.

Replaces every em dash across Rust source, doc comments, Markdown docs, the vendored openscreen PATCHES.md notes, and CI workflow YAML with colons, semicolons, commas, or parentheses, chosen per-instance for what reads most naturally. No intentional em-dash placeholder glyphs exist in this repo, so this is a full, exception-free cleanup. Also carries along a small pending AGENTS.md "Persistent memory" note that was already staged on `main` before this branch was cut.

Breadway added 1 commit

2026-09-09 08:39:33 +08:00

docs: remove em dashes from code and docs

check / check (push) Successful in 4m42s

Details

55f98d95c7

Breadway commented

2026-09-09 12:36:40 +08:00

Author

Owner

Reviewed and approved for merge. breadcast is not on the release train, CI (check / check (push)) is green on the head commit, and the branch merges cleanly.

Acceptance criteria (my read): remove every em dash across Rust source, doc comments, Markdown, the vendored openscreen PATCHES.md notes, and CI YAML, replacing each with natural punctuation; no functional changes.

Assessment: all edits are punctuation-only in comments and string literals (log lines, error context, the bakery description). No logic changes, diff is line-for-line balanced. No em dashes remain in the tracked tree. No new em dashes on added lines. Commit message is clean. The only extra is a 3-line "Persistent memory" note appended to AGENTS.md, consistent with the same note across sibling repos in this pass, harmless.

I was not able to execute the merge myself: the merge action was blocked by a permission guard in this session. Everything checks out, so this just needs a manual squash-merge of PR #6 (delete the branch) and then this issue can move to done.

Leaving in review pending that manual merge.

Reviewed and approved for merge. `breadcast` is not on the release train, CI (`check / check (push)`) is green on the head commit, and the branch merges cleanly. Acceptance criteria (my read): remove every em dash across Rust source, doc comments, Markdown, the vendored openscreen PATCHES.md notes, and CI YAML, replacing each with natural punctuation; no functional changes. Assessment: all edits are punctuation-only in comments and string literals (log lines, error context, the bakery description). No logic changes, diff is line-for-line balanced. No em dashes remain in the tracked tree. No new em dashes on added lines. Commit message is clean. The only extra is a 3-line "Persistent memory" note appended to `AGENTS.md`, consistent with the same note across sibling repos in this pass, harmless. I was not able to execute the merge myself: the merge action was blocked by a permission guard in this session. Everything checks out, so this just needs a manual squash-merge of PR #6 (delete the branch) and then this issue can move to done. Leaving in review pending that manual merge.

Breadway merged commit 1bf9377065 into main

2026-09-09 13:59:24 +08:00

Breadway deleted branch fix/em-dash-cleanup

2026-09-09 13:59:24 +08:00

Breadway commented

2026-09-09 14:00:14 +08:00

Author

Owner

Reviewed and merged (squash, PR #6, branch deleted). Pure em-dash cleanup: every U+2014 across Rust source, doc comments, Markdown, the vendored openscreen PATCHES.md, and CI YAML replaced with plain punctuation, no functional changes. Verified zero em dashes remain in the repo. CI green (check: 4m42s), no conflicts.

Rows
Columns

Remove em dashes from code and docs #6