Skip to content

Commit 476081e

Browse files
cursoragentmsukkari
andcommitted
docs: add requirement to run tests and build before pushing
Co-authored-by: Michael Sukkarieh <msukkari@users.noreply.github.com>
1 parent cee5e31 commit 476081e

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

CLAUDE.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -278,6 +278,10 @@ PR description:
278278
- If a GitHub issue number was provided, include `Fixes #<github_issue_number>` in the PR description
279279
- If a Linear issue ID was provided (e.g., SOU-123), include `Fixes SOU-123` at the top of the PR description to auto-link the PR to the Linear issue
280280

281+
Before pushing:
282+
- ALWAYS run `yarn test` and `yarn workspace @sourcebot/web build` before pushing to ensure tests pass and the build succeeds
283+
- Do not push code that fails tests or breaks the build
284+
281285
After the PR is created:
282286
- Update CHANGELOG.md with an entry under `[Unreleased]` linking to the new PR. New entries should be placed at the bottom of their section.
283287
- Do NOT add a CHANGELOG entry for documentation-only changes (e.g., changes only in `docs/`)

0 commit comments

Comments
 (0)