Commit 883f9ac
committed
docs(docs-audit): record why CHANGELOG.md is NOT added to the drift exclusion
#4091 dropped test files before deriving the changed-package roots, because a
test cannot make an implementation-accuracy doc stale. `packages/*/CHANGELOG.md`
is the obvious next candidate by the same argument — release notes define
behaviour no more than a test does — and extending the exclusion there is a
provable no-op. Measured before writing anything down:
- `chore: version packages` is the only PR class that mass-touches those files,
and it runs no GitHub Actions at all. `changesets/action` opens it with the
repo's GITHUB_TOKEN, and GitHub does not create workflow runs from
GITHUB_TOKEN-authored events. #3910 carries exactly one check run, from
Vercel's own app — `ci.yml` matches that PR unconditionally and still never
fired. So the drift gate never sees a release PR to be noisy on.
- Even if it did, `changeset version` writes `package.json` beside every
`CHANGELOG.md` it appends to (45 against 46 on page 1 of #3910's diff), so
dropping the CHANGELOGs would leave the derived package-root set identical.
The bump is not unverified, just verified later: `ci.yml` and `lint.yml` both run
on `push: main`, and `release.yml` gates publish on a green build — worth stating
so the absent PR checks are not read as a hole.
Recorded in the README beside the exclusion it neighbours, so the next reader
does not spend a PR rediscovering it.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QoA8AV99Ss1RRkLLAYmDzq1 parent 4f30943 commit 883f9ac
2 files changed
Lines changed: 36 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
41 | 41 | | |
42 | 42 | | |
43 | 43 | | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
44 | 62 | | |
45 | 63 | | |
46 | 64 | | |
| |||
0 commit comments