Skip to content

Add Windows build to CI#14

Draft
charlesw-salesforce wants to merge 5 commits into
mainfrom
cw/fix-vitest-axe-windows
Draft

Add Windows build to CI#14
charlesw-salesforce wants to merge 5 commits into
mainfrom
cw/fix-vitest-axe-windows

Conversation

@charlesw-salesforce
Copy link
Copy Markdown
Contributor

What does this PR do?

Updates vitest-env.d.ts to use imports to fix #13.

Replace /// <reference types="vitest-axe/extend-expect" /> with an
explicit declare module augmentation to fix TypeScript build failures
on Windows where the reference directive fails to resolve.
Replace /// <reference types> directives with import statements for
cross-platform compatibility. Add Windows CI job to catch platform-
specific type resolution issues.
@charlesw-salesforce charlesw-salesforce self-assigned this May 11, 2026
@charlesw-salesforce charlesw-salesforce marked this pull request as draft May 11, 2026 22:49
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salesforce Code Analyzer found 0 violations, including 0 in files changed by this pull request. See job summary page.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salesforce Code Analyzer found 0 violations, including 0 in files changed by this pull request. See job summary page.

@charlesw-salesforce charlesw-salesforce changed the title Fix Vitest Axe Issue Add Windows build to CI May 11, 2026
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salesforce Code Analyzer found 0 violations, including 0 in files changed by this pull request. See job summary page.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salesforce Code Analyzer found 0 violations, including 0 in files changed by this pull request. See job summary page.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

npm run build fails with numerous `error TS2339: Property 'toHaveNoViolations' does not exist on type 'JestMatchers<AxeResults>'

1 participant