Skip to content

Commit 706716d

Browse files
committed
wip
1 parent 6d41770 commit 706716d

File tree

1 file changed

+17
-7
lines changed

1 file changed

+17
-7
lines changed

CLAUDE.md

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -51,24 +51,34 @@ Based on the above, rate as:
5151
Alpine.js is a monorepo with packages in `/packages/`:
5252
- Each package has its own package.json
5353
- Build outputs go to `dist/` with `.cjs.js`, `.esm.js`, and `.min.js` versions
54-
- Tests use Jest framework
54+
- Browser tests use Cypress, unit tests use Vitest
5555
- CI runs on GitHub Actions
5656

5757
## Common Commands
5858

5959
```bash
60+
# Build
61+
npm run build # Build all packages
62+
63+
# Browser tests (Cypress)
64+
npm test # Run all tests
65+
npx cypress run --spec ./tests/cypress/integration/[filename].spec.js # Run single spec
66+
67+
# Unit tests (Vitest)
68+
npx vitest run tests/vitest/[filename].spec.js # Run single spec
69+
6070
# Review PRs
61-
gh pr list # List open PRs
71+
gh pr list # List open PRs
6272
gh pr view [number] # View PR details
6373
gh pr diff [number] # View code changes
6474
gh pr checks [number] # Check CI status
65-
gh api repos/alpinejs/alpine/pulls/[number]/comments # View comments
66-
67-
# Testing
68-
npm test # Run all tests
69-
npm run build # Build all packages
7075
```
7176

77+
## Manual Testing
78+
79+
1. Edit `./index.html` at project root
80+
2. Open in browser at `http://alpine.test/` (assumes local dev server mapped to directory name)
81+
7282
## Summary
7383

7484
After assessing the pull request on the above qualities, provide a summary explaining the problem this PR addresses and the fix, and why it's a good or bad fix. Do it in plain language as if you are personally advising me on what the PR is and weather or not I should merge it. And if not, what might need to be addressed first. If things need to be addressed, offer to address them yourself.

0 commit comments

Comments
 (0)