Skip to content

Punctuation changes - #202

Merged
ben-dev-au merged 1 commit into
mainfrom
chore/dash-sweep
Oct 11, 2026
Merged

ben-dev-au merged 1 commit into
mainfrom
chore/dash-sweep

Conversation

@ben-dev-au

Copy link
Copy Markdown
Owner

Replace every em and en dash in the repo with plain punctuation, in one pass now that the in-flight branches have merged.

What changed

  • 397 files; no em or en dash remains.
  • Comments, docstrings and docs: a dash became a colon, semicolon, comma or parentheses; ranges read "1 to 1000"; an en dash between words became a hyphen.
  • No code changed: every changed Python file has an identical syntax tree with string contents blanked; every changed YAML file still parses.
  • No literal dash is handled as data anywhere, so nothing needed escaping.

Visible strings worth a glance

  • .github/workflows/release.yml: step name is now Guard (release tag matches pyproject version).
  • .github/workflows/quarterly-review.yml: new issues are titled "Quarterly dependency review: "; nothing searches for the old title.
  • tests/test_settings_p3_visual.py: sample text reads "Result limit (1 to 1000): ...".
  • Some test assertion messages; no test compares them.

Checks

  • make batch-close on this tree: 5,995 passed, 3 skipped; all 12 app scenarios passed.
  • 397 files is past CodeRabbit's limit, so it will skip this PR; punctuation only, so that review is not needed here.

Comments, docstrings, test messages and workflow text now use a comma,
colon, semicolon or parentheses; ranges read "10 to 20". No code
changed: every Python file's AST is identical with string contents
blanked, and the changed YAML still parses.
@coderabbitai

coderabbitai Bot commented Oct 11, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: e267efc6-2861-4a19-b2d2-b88159ece434

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ben-dev-au ben-dev-au changed the title Replace em and en dashes with plain punctuation Punctuation changes. Oct 11, 2026
@ben-dev-au ben-dev-au changed the title Punctuation changes. Punctuation changes Oct 11, 2026
@ben-dev-au
ben-dev-au merged commit 266379a into main Oct 11, 2026
22 checks passed
@ben-dev-au
ben-dev-au deleted the chore/dash-sweep branch October 11, 2026 11:27
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