Skip to content

chore: sync ts-testing-all conventions - #16

Merged
bvandrc merged 2 commits into
mainfrom
chore/sync-ts-testing-all
Sep 27, 2026
Merged

bvandrc merged 2 commits into
mainfrom
chore/sync-ts-testing-all

Conversation

@bvandrc

@bvandrc bvandrc commented Sep 27, 2026

Copy link
Copy Markdown
Owner

Requested by Blake · project thread

Before: bvandrc-conventions#19 moved the testing rules shared by every suite into a new ts-testing-all.md, but this repo's sync workflow didn't list it, so those rules would have dropped out on the next sync.

After: the sync keeps conventions/ts-testing-all.md current, and CLAUDE.md imports it alongside the other convention files.

How: added ts-testing-all.md to files: in .github/workflows/sync-conventions.yml, ahead of the suite-specific files that build on it, and added its @conventions/ line to CLAUDE.md. The markdown files under conventions/ are copied from upstream main, the same content the next sync would bring, so the rules that moved out of the other files aren't missing in the meantime.

🤖 Generated with Claude Code

https://claude.ai/code/session_01GrSSi8k4q48PmHCXCMFVrN


Generated by Claude Code

The repo has a TypeScript test suite, so it syncs the testing rules shared across suites. Also brings the already-synced convention files up to date with upstream.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GrSSi8k4q48PmHCXCMFVrN
@github-actions

Copy link
Copy Markdown
Contributor

PR Diff Line Count

code comment

${\color{#2da44e}+}$

${\color{#bf8700}\sim}$

${\color{#e5534b}-}$

${\color{#2da44e}+}$

${\color{#e5534b}-}$

Docs

${\color{#2da44e}36}$

${\color{#bf8700}2}$

${\color{#e5534b}4}$

${\color{#848d97}0}$

${\color{#848d97}0}$

Config

${\color{#848d97}0}$

${\color{#bf8700}1}$

${\color{#848d97}0}$

${\color{#848d97}0}$

${\color{#848d97}0}$

Total

${\color{#2da44e}36}$

${\color{#bf8700}3}$

${\color{#e5534b}4}$

${\color{#848d97}0}$

${\color{#848d97}0}$

GitHub reports ${\color{#2da44e}+41}$ / ${\color{#e5534b}-7}$

~ is a line changed in place — cloc counts it once rather than as an add plus a delete, so these columns do not sum to GitHub's.
Blank lines are excluded above: ${\color{#2da44e}+2}$ / ${\color{#848d97}-0}$.

@bvandrc bvandrc self-assigned this Sep 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Size Change: 0 B

Total Size: 1.42 kB

ℹ️ View Unchanged
Filename Size
dist/index.js 1.42 kB

compressed-size-action

Matches the order the bvandrc-conventions README recommends.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GrSSi8k4q48PmHCXCMFVrN
@bvandrc
bvandrc merged commit efc89bc into main Sep 27, 2026
5 checks passed
@bvandrc
bvandrc deleted the chore/sync-ts-testing-all branch September 27, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants