Skip to content

Commit 28b817a

Browse files
dbrattliclaude
andauthored
ci: tighten PR scope rules to enforce js/ts as one unit (#4570)
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 4f86cbe commit 28b817a

2 files changed

Lines changed: 2 additions & 8 deletions

File tree

.github/workflows/conventional-pr-title.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,10 +21,4 @@ jobs:
2121
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
2222
with:
2323
scopes: |
24-
js
25-
ts
26-
dart
27-
python
28-
rust
29-
beam
30-
js/ts
24+
^(?:all|(?:js/ts|dart|python|rust|beam)(?:/(?:js/ts|dart|python|rust|beam))*)$

.github/workflows/repo-assist.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -301,7 +301,7 @@ Update memory with labels applied and cursor position.
301301
e. Add a test for the bug if feasible.
302302
f. DO NOT update the changelogs, they are automatically generated based on the commit history
303303
g. Create a draft PR with: AI disclosure, `Closes #N`, root cause, fix rationale, and trade-offs. CI will validate the build and tests.
304-
The PR title should follow the Conventional Commits format (e.g., `fix: ...`, `feat: ...`, `perf: ...`) with a target prefix if applicable (e.g., `fix(python): ...`, `feat(js): ...`).
304+
The PR title should follow the Conventional Commits format (e.g., `fix: ...`, `feat: ...`, `perf: ...`) with a target scope if applicable. Allowed scopes: `js/ts`, `dart`, `python`, `rust`, `beam`, `all`. JS and TS must always be combined as `js/ts` (never used separately). Multi-target changes use slash-separated combinations such as `fix(js/ts/python): ...` or `fix(python/beam): ...`. Use `all` for changes affecting every target, and omit the scope entirely for non-target-specific changes.
305305
h. Post a single brief comment on the issue linking to the PR.
306306
3. Update memory with fix attempts and outcomes.
307307

0 commit comments

Comments
 (0)