Recast clause-joining semicolons in WORKFLOW.md (prose style sweep) - #386
Merged
Conversation
WORKFLOW.md pervasively joined independent clauses with semicolons, which the AGENTS.md prose rule bans (a semicolon may separate list items that themselves contain commas, not join two clauses). Recast 83 such semicolons - to a period where both sides are full sentences, to a comma plus conjunction where they flow, or to an ASCII dash for a trailing aside - preserving the exact meaning and every inline-code span, link, and token. Left intact: semicolons that separate comma-bearing list items (the rule's explicit exception), semicolons inside code and links, and a few telegraphic parenthetical noun-pairs and trace-table cells that are not two independent clauses. Surfaced by the develop->main promotion review (#383). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR performs a prose-style sweep in WORKFLOW.md to align the document with the repository's prose rule that forbids using semicolons to join independent clauses, while preserving meaning and technical terminology.
Changes:
- Replaced clause-joining semicolons with periods, comma + conjunction, or an ASCII hyphen for asides where appropriate.
- Kept semicolons where they function as valid separators (e.g., within structured/telegraphic content or non-clause usage) and avoided altering inline code/link tokens.
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
WORKFLOW.md pervasively joined independent clauses with semicolons (~80 prose lines), which the AGENTS.md prose rule bans (a semicolon may separate list items that themselves contain commas, but must not join two clauses). Surfaced by the
develop→mainpromotion review (#383).What changed
, and/, but), or an ASCII-for a trailing aside (2). Meaning preserved exactly; every inline-code span, link, and token intact.Verification
markdownlint clean; CRLF preserved; every changed line audited for meaning preservation and comma-splice avoidance.
This is the first of the carried contract docs (AGENTS.md and CODESTYLE.md have the same pattern, tracked separately). It also carries to the fleet, so it will re-vendor downstream.
🤖 Generated with Claude Code