Skip to content

Remove em dashes and en dashes from prose#17

Merged
gek0z merged 2 commits intomainfrom
chore/remove-em-dashes
Apr 15, 2026
Merged

Remove em dashes and en dashes from prose#17
gek0z merged 2 commits intomainfrom
chore/remove-em-dashes

Conversation

@gek0z
Copy link
Copy Markdown
Owner

@gek0z gek0z commented Apr 15, 2026

Summary

  • Mechanical replacement of 102 em dashes and 1 en dash across docs, comments, and user-facing strings.
  • Follow-up polish pass to avoid awkward commas where a colon, semicolon, or period fit better: label/definition bullets (README features, troubleshooting, release pipeline; CONTRIBUTING project layout), comma-splices in SECURITY.md, user-facing error/banner strings in YouTubeService.swift, ImportSessionWindow.swift, and SettingsView.swift.
  • Version-display fallback in SettingsView.versionDisplay now uses · to match the other branch of the same ternary.

Test plan

  • swift-format lint --strict passes (pre-commit hook ran clean).
  • README, CONTRIBUTING, SECURITY, NOTICE render without awkward orphan commas.
  • User-facing strings (sign-in banner, Keychain banner, disconnect error, settings blurb) read as complete sentences.

Replaced all em/en dashes across docs, comments, and user-facing strings
with colons, semicolons, periods, or commas depending on context. Label
/definition pairs (README bullets, troubleshooting entries, release
pipeline steps, CONTRIBUTING project layout) use colons; comma-splices
use semicolons or periods. User-visible strings split into separate
sentences where the dash was a sentence break.
The reference existed to contrast with the old Data-API-v3 path; now
that the app only uses InnerTube, surfacing "no Google Cloud setup
needed" invites more confusion than it resolves. Trimmed to the
essentials.
@gek0z gek0z added the run-ci Run the CI workflow on this PR label Apr 15, 2026
@gek0z gek0z merged commit 9a702d9 into main Apr 15, 2026
2 checks passed
@gek0z gek0z deleted the chore/remove-em-dashes branch April 15, 2026 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-ci Run the CI workflow on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant