Skip to content

Convert to vitest #43

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.

Sign up for GitHub

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

Draft
TylerJDev wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Convert to vitest #43

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
eada17d
Convert to vitest
TylerJDev Aug 13, 2025
0a98202
Change workflow to use node 18
TylerJDev Aug 15, 2025
9147fab
Fix tests, add `happy-dom`
TylerJDev Aug 15, 2025
c1eb316
Eslint edits
TylerJDev Aug 15, 2025
135aa5d
Apply suggestion from @Copilot
TylerJDev Aug 15, 2025
5f05e0d
Fix yml error
TylerJDev Aug 15, 2025
cdddb66
Remove duplicate `run`
TylerJDev Aug 15, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Remove duplicate run
  • Loading branch information
@TylerJDev
TylerJDev committed Aug 15, 2025
commit cdddb66ee11667b91b85bbeb301fce38123f2b28
3 changes: 0 additions & 3 deletions .github/workflows/nodejs.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,9 +10,6 @@ jobs:
with:
node-version: 18.x
- name: npm install, build, and test
run: |
npm install
npm run build --if-present
run: |
npm install
npm run build --if-present
Expand Down