Skip to content

docs(claude): replace em-dashes in CLAUDE.md - #2255

Merged
cristim merged 1 commit into
mainfrom
docs/claude-md-no-emdash
Oct 17, 2026
Merged

cristim merged 1 commit into
mainfrom
docs/claude-md-no-emdash

Conversation

@cristim

@cristim cristim commented Oct 17, 2026

Copy link
Copy Markdown
Member

Closes #1295

Summary

CLAUDE.md contained 12 em-dashes (U+2014), which break the project rule against them. This PR replaces each one with whatever punctuation suits its sentence: a colon, commas, parentheses or a full stop. Two headings and a comment in the label-mirroring bash block are among the 12. The wording is otherwise unchanged, and no other file in the repo contains em-dashes.

Evidence

  • grep -c "—" CLAUDE.md returns 0.
  • markdownlint and the pre-commit hooks pass.
  • A git diff --word-diff shows only the em-dash tokens and the punctuation next to them changed.
  • Nothing links to the renamed headings: git grep finds no anchor references to them outside CLAUDE.md.
  • An independent non-author reviewer (sonnet, high) did two passes and found no actionable findings at 1f3981b.

🤖 Generated with Claude Code

Replace em-dashes with colons, commas, semicolons or parentheses per the
no-em-dash rule.

Closes #1295

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cristim cristim added priority/p3 Polish / idea / may never ship severity/low Minor harm urgency/eventually No deadline impact/internal Team-internal only effort/s Hours type/docs Documentation triaged Item has been triaged labels Oct 17, 2026
@coderabbitai

coderabbitai Bot commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

  • Run on-demand review

This review includes 1 billable file and costs up to $0.25.

  • Ask an admin to make reviews automatic

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Or wait 34 minutes for your next included review.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 73 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: LeanerCloud/cloud-commitments-cli/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: c064d5a6-d1ed-4968-b35c-cf25f98f85fd
📥 Commits

Reviewing files that changed from the base of the PR and between ee2b597 and 1f3981b.

📒 Files selected for processing: 1
  • CLAUDE.md
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@cristim

cristim commented Oct 17, 2026

Copy link
Copy Markdown
Member Author

Independent review verdict: CLEAN at 1f3981b

Reviewer: fresh Claude agent (Sonnet 5.5, high effort), not the author, briefed cold. Coordinator: claude-cc-reviewer.

Blocking findings: none.

What was checked:

  • Punctuation only. All 12 changed lines (15, 70, 188, 194, 211, 249, 261, 301, 304, 341, 344) swap an em-dash for :, ,, parentheses or ;. The meaning is unchanged. The review gate, the MANDATORY rules and the planning loop are intact.
  • Code blocks. The only edit inside a fenced block is a bash comment (# Guard against empty $LABELS: ...). No command changed, and the fences are intact.
  • Em-dashes gone. grep -c $'—' CLAUDE.md at this head returns 0, which meets HYG: Replace 11 pre-existing em-dashes in CLAUDE.md (project rule) #1295.
  • Renamed headings. Three headings changed. Nothing on origin/main links to their old anchors (#issue-assignment, #post-push-ci-watcher, #pr-labeling).

Local verification: non-runtime change. I checked the rendered markdown and the anchors as described above.

Gate:

  • Labels are mirrored from HYG: Replace 11 pre-existing em-dashes in CLAUDE.md (project rule) #1295.
  • CodeRabbit is rate limited and has posted no reviews or inline comments. Merge proceeds under CR waived: quota, adversarial review + local verification + green CI, and a retrospective CR review is needed once quota returns.
  • Merge waits for CI to be green at this SHA.

@cristim
cristim merged commit b0eeb46 into main Oct 17, 2026
12 checks passed
@cristim
cristim deleted the docs/claude-md-no-emdash branch October 17, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort/s Hours impact/internal Team-internal only priority/p3 Polish / idea / may never ship severity/low Minor harm triaged Item has been triaged type/docs Documentation urgency/eventually No deadline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HYG: Replace 11 pre-existing em-dashes in CLAUDE.md (project rule)

1 participant