Skip to content

fix(vr-tests-web-component): explicitly add @types/web and specify those within types globals in order to pass tsc#31451

Merged
Hotell merged 1 commit into
microsoft:masterfrom
Hotell:wc-3-followup/fix-vr-test-type-check-issues
May 22, 2024
Merged

fix(vr-tests-web-component): explicitly add @types/web and specify those within types globals in order to pass tsc#31451
Hotell merged 1 commit into
microsoft:masterfrom
Hotell:wc-3-followup/fix-vr-test-type-check-issues

Conversation

@Hotell

@Hotell Hotell commented May 22, 2024

Copy link
Copy Markdown
Contributor

Previous Behavior

Master pipeline fails because type issues with vr-tests-web-components

image

New Behavior

Because web-components use https://www.npmjs.com/package/@types/web package in order to override DOM lib types every app and library within monorepo that uses wc as dependency needs to specify web within tsconfig types global

Related Issue(s)

…ose within types globals in order to pass tsc
@Hotell Hotell changed the title fix(vr-tests-web-component): explicitly add @types/web and specify th… fix(vr-tests-web-component): explicitly add @types/web and specify those within types globals in order to pass tsc May 22, 2024
@github-actions github-actions Bot added this to the April Project Cycle Q1 2024 milestone May 22, 2024
@fabricteam

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

✅ No changes found

Comment thread apps/vr-tests-web-components/package.json
@codesandbox-ci

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@Hotell Hotell marked this pull request as ready for review May 22, 2024 11:58
@Hotell Hotell requested review from a team as code owners May 22, 2024 11:58
@Hotell Hotell merged commit 1e2ce79 into microsoft:master May 22, 2024
@Hotell Hotell deleted the wc-3-followup/fix-vr-test-type-check-issues branch May 22, 2024 12:02
@Hotell

Hotell commented May 22, 2024

Copy link
Copy Markdown
Contributor Author

admin merged with v-build approval in order to unblock master

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants