docs(skill): capitalize the word after a changeset audience label - #235
Conversation
… label The writing-changesets skill didn't state that the sentence after a bold `**Audience:**` label must start with a capital letter, and its own examples were inconsistent (some lowercase). Add an explicit rule with ✅/❌ examples, including the identifier-initial case (reword so a lowercase code identifier isn't sentence-initial), and fix the three lowercase openers in the skill's own example changesets. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
Next review available in: 58 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
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. Comment |
|
🚅 Deployed to the ePDS-pr-235 environment in ePDS
|
There was a problem hiding this comment.
Pull request overview
Updates the writing-changesets agent skill documentation to make per-audience prose style consistent by requiring capitalization immediately after audience labels, with examples and guidance for awkward identifier-led sentences.
Changes:
- Adds an explicit “capitalize after audience label” rule with ✅/❌ examples, including guidance for lowercase code identifiers.
- Clarifies that the capitalization rule applies per bullet when per-audience details are written as bullet lists.
- Fixes lowercase sentence starters in the skill’s own example changesets.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Coverage Report for CI Build 30952044983Warning Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes. Coverage increased (+0.2%) to 57.909%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions51 previously-covered lines in 3 files lost coverage.
Coverage Stats
💛 - Coveralls |
Two fixes to the capitalization rule added in d897e61. The rule referred to a literal `**Audience:**` label, which does not exist — the real labels are `**End users:**`, `**Client app developers:**` and `**Operators:**`. A reader could reasonably have taken `**Audience:**` for a valid label. Name two real ones instead. The paragraph also carried a blank line before its bullet list, which `prettier --check` rejected and which is inconsistent with the two other nested bullet lists in this file (neither has one). Raised by Copilot in review of #235. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
The
writing-changesetsskill didn't state that the sentence after a bold**Audience:**label must start with a capital letter — the colon is a lead-in, not a mid-sentence break. Its own examples were inconsistent (some lowercase openers).Changes:
email_verifiedclaim …", not "email_verifiedis now …").two→Two,no→No×2).Prompted by review feedback on #234's changeset (
apps/the/deploy→Apps/The/Deploy); those suggestions are applied to #234 separately.🤖 Generated with Claude Code