Skip to content

Fix Vitest run mode in CI#34

Open
stevenschling13 wants to merge 1 commit intomainfrom
codex/fix-ci-workflow-failures
Open

Fix Vitest run mode in CI#34
stevenschling13 wants to merge 1 commit intomainfrom
codex/fix-ci-workflow-failures

Conversation

@stevenschling13
Copy link
Owner

Summary

  • run Vitest in run mode so the test script exits cleanly
  • update the CI workflow to rely on the new test command

Testing

  • pnpm test
  • pnpm lint
  • pnpm typecheck

https://chatgpt.com/codex/tasks/task_e_6907851bbd78832c9a7a4b1f20ccfb56

Copilot AI review requested due to automatic review settings November 2, 2025 16:24
Copy link

Copilot AI left a 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 standardizes the test command to run in non-watch mode by default. The change ensures that pnpm test consistently executes tests once and exits, which is appropriate for both local development and CI environments.

Key Changes:

  • Modified the test script to use vitest run instead of vitest (watch mode)
  • Simplified the CI workflow to use pnpm test directly instead of passing --run flag

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Updated test script to use vitest run for non-watch mode execution
.github/workflows/ci.yml Simplified CI test command to use the updated test script

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant