ci: run integration tests in PR checks#198
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View your CI Pipeline Execution ↗ for commit a46fe73
☁️ Nx Cloud last updated this comment at |
commit: |
🎯 Changes
test:integrationwas wired intotest:ciandnx.jsonback in #148 but was later dropped fromtest:prin #169 ("rm integration tests from test:pr"), so PR CI silently skipped packages/intent/tests/integration/** on every PR since. This restores it: pr.yml now runspnpm run test:integrationas its own step, and a matching roottest:integrationscript (nx affected --targets=test:integration --exclude=examples/**) is added for local use, mirroringtest:lib.✅ Checklist
pnpm run test:pr.🚀 Release Impact