You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
yarn test currently builds the package and runs node:test files against dist.
The existing tests work, but there is no pull request workflow that runs them.
As more CODEOWNERS and CLI regressions are covered, testing TypeScript source directly would make the suite easier to develop.
Proposed work
Add Vitest and its required Vite dependency.
Migrate the existing CODEOWNERS tests to Vitest without losing their assertions.
Make yarn test run the suite once and return a failing exit code when a test fails.
Add a pull request workflow that installs dependencies with yarn install --immutable and runs yarn test, yarn typecheck, and yarn build.
Acceptance criteria
yarn test passes from a clean checkout without prebuilt dist files.
Context
yarn test currently builds the package and runs node:test files against dist.
The existing tests work, but there is no pull request workflow that runs them.
As more CODEOWNERS and CLI regressions are covered, testing TypeScript source directly would make the suite easier to develop.
Proposed work
Acceptance criteria