Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(recipes-react-components): explicitly set types to not include whole @types/* globals which are cauising issues with addition of @types/web #31463

Merged
merged 1 commit into from
May 23, 2024

Conversation

Hotell
Copy link
Contributor

@Hotell Hotell commented May 23, 2024

Previous Behavior

New Behavior

  • see PR title
  • it turns out the culprit is that recipes app didn't set types config property. if that's not present typescript loads all @type/* globals which is a antipattern and started to be problematic since @types/web landed to master with web-components v3 ( which is hoisted thus getting those node_modules issues on CI )

Related Issue(s)

…ole @types/* globals which are cauising issues with addition of @types/web
@github-actions github-actions bot added this to the April Project Cycle Q1 2024 milestone May 23, 2024
@Hotell Hotell changed the title fix(recipes-react-components): explicitly set types to not include wh… fix(recipes-react-components): explicitly set types to not include whole @types/* globals which are cauising issues with addition of @types/web May 23, 2024
@fabricteam
Copy link
Collaborator

📊 Bundle size report

✅ No changes found

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 23, 2024 13:38
@Hotell Hotell requested review from a team and sopranopillow as code owners May 23, 2024 13:38
@Hotell Hotell merged commit 66ce207 into microsoft:master May 23, 2024
19 checks passed
@Hotell Hotell deleted the fix/type-checking-issue-fix-v2 branch May 23, 2024 13:40
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request May 28, 2024
* master: (49 commits)
  Update focus order in sankey chart for vertical navigation (microsoft#31469)
  chore: use new performant 'type-check' for v9 libaries (microsoft#31454)
  applying package updates
  fix(Timepicker-compat): clearIcon not working in freeform (microsoft#31324)
  chore: re-enable lint rule (microsoft#31459)
  feat(react-tag-picker): adds text property to TagPickerOption (microsoft#31474)
  feat(recipes): create package with initial implementation (moved from /apps) (microsoft#31386)
  applying package updates
  applying package updates
  chore: revert globals changes (microsoft#31470)
  (web-components) Use `ElementInternals` for TextInput elements (microsoft#31201)
  chore:(docs) Update and migrate component implementation guide (microsoft#31398)
  disallow all globals in Fluent v9 (microsoft#30967)
  chore:(react-nav-preview) Recomposing more components and some pixel pushing (microsoft#31387)
  fix(pr-deploy-site): explicitly set types to not include whole @types/* globals which are causing issues with addition of @types/web (microsoft#31465)
  fix(recipes-react-components): explicitly set types to not include whole @types/* globals which are cauising issues with addition of @types/web (microsoft#31463)
  applying package updates
  applying package updates
  applying package updates
  Update IconDirectionContextProvider import to import from specific path (microsoft#31006)
  ...
miroslavstastny pushed a commit to miroslavstastny/fluentui that referenced this pull request Jun 14, 2024
…ole @types/* globals which are cauising issues with addition of @types/web (microsoft#31463)
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.

3 participants