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 intomicrosoft: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 intomicrosoft:masterfrom
Hotell:wc-3-followup/fix-vr-test-type-check-issues

Conversation

@Hotell
Copy link
Contributor

@Hotell Hotell commented May 22, 2024

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
Collaborator

📊 Bundle size report

✅ No changes found

@codesandbox-ci
Copy link

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
Copy link
Contributor Author

Hotell commented May 22, 2024

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