Skip to content

ci: run the test suite on pull requests - #310

Merged
edwinyjlim merged 1 commit into
PostHog:mainfrom
Peetee06:fix/add-ci-tests
Aug 3, 2026
Merged

ci: run the test suite on pull requests#310
edwinyjlim merged 1 commit into
PostHog:mainfrom
Peetee06:fix/add-ci-tests

Conversation

@Peetee06

@Peetee06 Peetee06 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Adds running vitest to the PostHog Context Mill - Build workflow so regressions get caught on PRs before merging.

🤖 Generated with Claude Code

The 18 vitest files never ran in CI. pnpm build only exercises the configs
that exist in-repo today, so it misses every parser guard and error path —
deleting the validateCommandName call from parseCliBlock leaves the build
green while 5 cli-block tests fail.

Placed before Build to fail fast: 0.7s for 139 tests, no secrets, no network,
so it runs on fork PRs too.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LpsYRXz97rxeTQwF1my87D
@Peetee06
Peetee06 marked this pull request as ready for review August 2, 2026 11:10
@Peetee06
Peetee06 requested a review from a team as a code owner August 2, 2026 11:10
@edwinyjlim
edwinyjlim merged commit bd6bdb5 into PostHog:main Aug 3, 2026
10 of 11 checks passed
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