Skip to content

chore(react-components): fix build:local task#22664

Merged
Hotell merged 5 commits into
microsoft:masterfrom
Hotell:hotell/build/fix-local-api-extractor
Apr 28, 2022
Merged

chore(react-components): fix build:local task#22664
Hotell merged 5 commits into
microsoft:masterfrom
Hotell:hotell/build/fix-local-api-extractor

Conversation

@Hotell

@Hotell Hotell commented Apr 27, 2022

Copy link
Copy Markdown
Contributor

Current Behavior

  • after migration of react-components to react-components parent folder build:local is broken

New Behavior

  • local:build works
  • move generator has been updated to accommodate this behaviour for other packages
  • updates react-components tsConfig override in api extractor config

Related Issue(s)

Fixes #21910

@Hotell Hotell marked this pull request as ready for review April 27, 2022 10:57
@Hotell Hotell requested review from a team as code owners April 27, 2022 10:57
@Hotell Hotell added this to the April Project Cycle Q1 2022 milestone Apr 27, 2022
@Hotell Hotell changed the title Hotell/build/fix local api extractor chore(react-components): fix build:local task Apr 27, 2022
@fabricteam

fabricteam commented Apr 27, 2022

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against f25dc32e6a374734b2326bae277a28021f44bc0e

@codesandbox-ci

codesandbox-ci Bot commented Apr 27, 2022

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.

Latest deployment of this branch, based on commit 71170a0:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@size-auditor

size-auditor Bot commented Apr 27, 2022

Copy link
Copy Markdown

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: f25dc32e6a374734b2326bae277a28021f44bc0e (build)

@fabricteam

fabricteam commented Apr 27, 2022

Copy link
Copy Markdown
Collaborator

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
FluentProviderWithTheme virtual-rerender-with-unmount 281 262 10 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 894 902 5000
Button mount 520 558 5000
FluentProvider mount 1891 1950 5000
FluentProviderWithTheme mount 291 288 10
FluentProviderWithTheme virtual-rerender 229 233 10
FluentProviderWithTheme virtual-rerender-with-unmount 281 262 10 Possible regression
MakeStyles mount 1588 1504 50000

@TristanWatanabe TristanWatanabe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for fixing this! I just realized that generator also needs to update the actual build:local script in the package json to account for the new path (i.e. this portion: ./dist/packages/react-avatar/src). I can make a follow up PR to fix this in the move-packages script

Comment thread tools/generators/move-packages/index.ts Outdated
@Hotell

Hotell commented Apr 27, 2022

Copy link
Copy Markdown
Contributor Author

kk pls (the additional update/fix) do, ty!

Comment thread tools/generators/move-packages/index.spec.ts
Comment thread tools/generators/move-packages/index.ts Outdated
@Hotell Hotell enabled auto-merge (squash) April 28, 2022 10:33
@Hotell Hotell merged commit a1dca96 into microsoft:master Apr 28, 2022
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
* chore(react-components): fix build:local task

* feat(tools): update api-extractor.local when running move generator

* generate change file

* fixup! feat(tools): update api-extractor.local when running move generator

* Update tools/generators/move-packages/index.ts

Co-authored-by: Tristan Watanabe <[email protected]>

Co-authored-by: Tristan Watanabe <[email protected]>
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.

tsconfig override in packages/react-components/config/api-extractor.local.json is out of date with updates to main tsconfig

4 participants