Skip to content

Commit d7e5fc9

Browse files
authored
Merge pull request #40 from hotdata-dev/docs/comment-style-harness
docs(review): add the comment style harness
2 parents 5f4dccf + a9c4fd7 commit d7e5fc9

6 files changed

Lines changed: 558 additions & 0 deletions

File tree

.github/workflows/tests.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,12 @@ jobs:
5555
- name: Context step end to end
5656
run: tests/context-step-test.sh
5757

58+
# Guards the comment style harness's corpus rather than the review workflow: the
59+
# recorded totals in docs/comment-style-harness.md are only comparable while the
60+
# committed corpus keeps the shape they were measured against.
61+
- name: Comment style corpus
62+
run: tests/comment-style-corpus-test.sh
63+
5864
# Runs claude-pr-review.yml itself, from this commit, with the review step skipped. The checks
5965
# above all read the workflow as text; this one hands it to Actions and asks whether it starts.
6066
#

README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,10 @@ Automated code review on every pull request using [claude-code-action](https://g
1313
- **No issues** — silent approve
1414

1515
The review prompt lives in [`docs/claude-pr-review-prompt.md`](docs/claude-pr-review-prompt.md).
16+
Changes to its comment-style rules go through
17+
[`docs/comment-style-harness.md`](docs/comment-style-harness.md) first — the prompt deploys
18+
org-wide from `main` with no staging, and that procedure is the staging. It does not cover
19+
the severity or summary-comment rules; that document says why.
1620

1721
### Frontloaded review context
1822

0 commit comments

Comments
 (0)