-
Notifications
You must be signed in to change notification settings - Fork 0
set up some basic stuff in github actions #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR sets up basic testing infrastructure and GitHub Actions workflows for the project, including Vitest configuration, testing dependencies, a sample test, and CI/CD workflows for running tests and security audits.
Key Changes
- Added Vitest configuration with React and TypeScript support using jsdom environment
- Added GitHub Actions workflows for automated testing and security audits on all branches
- Created a reusable composite action for consistent Node.js and pnpm setup across workflows
Reviewed Changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| vitest.config.mts | New Vitest configuration with React plugin, TypeScript paths, and jsdom environment |
| package.json | Added test script, packageManager field, and testing dependencies (vitest, @testing-library/react, jsdom) |
| tests/page.test.tsx | Sample test for the Home page component using React Testing Library |
| .github/workflows/test.yml | CI workflow to run tests on push/PR to any branch |
| .github/workflows/npm-audit.yml | CI workflow to run pnpm security audit with moderate severity threshold |
| .github/actions/setup/action.yml | Reusable composite action for installing pnpm, Node.js, and project dependencies |
| pnpm-lock.yaml | Lock file updates reflecting new testing dependencies |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
71e62cb to
3b20f67
Compare
No description provided.