Skip to content

Conversation

@dmytrokirpa
Copy link
Contributor

see PR title

@dmytrokirpa dmytrokirpa requested a review from a team as a code owner August 11, 2025 11:00
@dmytrokirpa dmytrokirpa requested a review from Hotell August 11, 2025 11:00
@dmytrokirpa dmytrokirpa self-assigned this Aug 11, 2025
@dmytrokirpa dmytrokirpa force-pushed the tests/react-19-integration branch from d5cf345 to de20fbb Compare August 12, 2025 14:00
"preinstall": "node ./tools/scripts/preinstall.mjs",
"postinstall": "patch-package",
"dedupe": "npx yarn-deduplicate --strategy highest"
"dedupe": "npx yarn-deduplicate --strategy highest --exclude @types/react-dom @types/react"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Running npx yarn-deduplicate --strategy highest leads to this: https://github.com/microsoft/fluentui-contrib/pull/468/files#diff-51e4f558fae534656963876761c95b83b6ef5da5103c4adef6768219ed76c2deR6127-R6130.

This forces @types/[email protected] everywhere, causing pipeline failures. So far, adding excludes is the only workaround I've found. Open to better ideas.

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah, either detective work and trying resolutions on sub paths ( which might not work with current yarn ) or this. it's ok for now / once react integration tester CLI will land we will replace current approach which will also not cause this kind of issues

Copy link
Contributor

@Hotell Hotell Aug 18, 2025

Choose a reason for hiding this comment

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

note that excluding @types/react should be enough

image

Copy link
Contributor Author

@dmytrokirpa dmytrokirpa Aug 18, 2025

Choose a reason for hiding this comment

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

Unfortunately that doesn't work, removing @types/react-dom leads to:

Screenshot 2025-08-18 at 15 05 58

@dmytrokirpa dmytrokirpa requested a review from a team as a code owner August 12, 2025 16:40
@dmytrokirpa dmytrokirpa removed the request for review from a team August 14, 2025 12:15
@dmytrokirpa dmytrokirpa merged commit c7c8da1 into microsoft:main Aug 18, 2025
4 of 6 checks passed
@dmytrokirpa dmytrokirpa deleted the tests/react-19-integration branch August 18, 2025 13:53
marcosmoura added a commit to marcosmoura/fluentui-contrib that referenced this pull request Sep 8, 2025
…matic-dnd

* main: (211 commits)
  fix(docsite): add missing dependency for extract-storybook-llms target (microsoft#486)
  feat(react-19-tests): bootstrap react 19 integration tests (microsoft#468)
  feat(docsite): generate llms.txt (microsoft#476)
  docs(react-virtualizer): Update virtualizer stories to use the same storybook Meta format as contrib repo (microsoft#474)
  chore: bump react-components to latest and adjust types to support React 18 (microsoft#387)
  chore(deps-dev): bump @swc/core from 1.11.24 to 1.13.3 (microsoft#463)
  chore(deps-dev): bump patch-package from 7.0.0 to 7.0.2 (microsoft#465)
  chore(deps-dev): bump @swc/cli from 0.7.7 to 0.7.8 (microsoft#467)
  chore: migrate to nx 21.3.10 (microsoft#453)
  ci(dependabot): ignore major bumps (microsoft#461)
  chore(deps-dev): bump eslint-config-prettier from 10.1.5 to 10.1.8 (microsoft#460)
  chore(deps-dev): bump react-virtualized-auto-sizer and @types/react-virtualized-auto-sizer (microsoft#454)
  chore(deps): bump nrwl/nx-set-shas from 4.3.0 to 4.3.3 (microsoft#456)
  applying package updates
  feat(react-virtualizer): initial/stable release (microsoft#452)
  docs(gh): add issue templates (microsoft#442)
  feat(react-virtualizer): migrate package from fluentui core repo (microsoft#441)
  feat(react-gamepad-navigation): adding unit tests (microsoft#443)
  fix(react-gamepad-navigation): handling Gamepad API access denied (microsoft#416)
  fix(react-keytips): remove default event prevention in useHotkeys (microsoft#430)
  ...
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.

2 participants