-
Notifications
You must be signed in to change notification settings - Fork 1
Project update #46
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
Project update #46
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #46 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 1 1
Lines 62 57 -5
Branches 11 10 -1
=========================================
- Hits 62 57 -5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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 migrates the project's testing infrastructure from Jest to Vitest and updates its development dependencies and configuration files accordingly.
Key Changes:
- Migration from Jest to Vitest for testing framework
- Update of ESLint configuration to use flat config format with modern TypeScript ESLint
- Minimum Node.js version requirement raised from 14.20.1 to 20
Reviewed changes
Copilot reviewed 13 out of 16 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| vitest.setup.ts | New setup file configuring MongoDB URL from global setup via Vitest injection |
| vitest.global-setup.ts | New global setup file managing MongoDB Memory Server lifecycle for tests |
| vitest.config.ts | New Vitest configuration file replacing Jest config |
| test/migrations/1696851672239-test-migration.js | Updated to use environment variable for MongoDB URL instead of Jest globals |
| test/index.test.ts | Migrated test file from Jest to Vitest with updated assertions and test utilities |
| package.json | Updated dependencies and scripts to use Vitest instead of Jest, raised Node requirement |
| jest-mongodb-config.js | Removed Jest-specific MongoDB configuration (no longer needed) |
| eslint.config.mjs | New ESLint flat config replacing legacy .eslintrc.js |
| CHANGELOG.md | Added unreleased section documenting breaking change for Node 20 requirement |
| .npmignore | Simplified to whitelist approach instead of blacklist |
| .github/workflows/main.yml | Updated CI workflow to use Vitest and Node 20 |
| .eslintrc.js | Removed legacy ESLint configuration file |
| .eslintignore | Removed as no longer needed with flat config |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.