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

perf: exclude cypress test files from tsc Program for .d.ts generation and type-checking #30989

Conversation

Hotell
Copy link
Contributor

@Hotell Hotell commented Apr 8, 2024

Pre-requirements

Previous Behavior

  • we have invalid setup for Type checking and cypress component tests in various projects ( probably caused by not invoking cy config generator )

New Behavior

  • all projects using cypress are properly configured
  • .d.ts generation now avoids running tsc on cypress file trees which adds some speed boost

Related Issue(s)

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 8, 2024

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
FluentProviderWithTheme virtual-rerender 31 30 10 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 620 614 5000
Button mount 301 308 5000
Field mount 1121 1185 5000
FluentProvider mount 686 712 5000
FluentProviderWithTheme mount 81 79 10
FluentProviderWithTheme virtual-rerender 31 30 10 Possible regression
FluentProviderWithTheme virtual-rerender-with-unmount 75 70 10
MakeStyles mount 848 828 50000
Persona mount 1766 1704 5000
SpinButton mount 1330 1413 5000

Copy link

codesandbox-ci bot commented Apr 8, 2024

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.

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 8, 2024

📊 Bundle size report

✅ No changes found

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 8, 2024

🕵 fluentuiv9 No visual regressions between this PR and main

@Hotell Hotell force-pushed the type-checking-perf/prerequirements/fix-cypress-tsc-slowdonws branch from 3eae6f9 to 86b5cdc Compare April 8, 2024 13:55
@Hotell Hotell marked this pull request as ready for review April 8, 2024 14:27
@Hotell Hotell requested review from a team, sopranopillow and khmakoto as code owners April 8, 2024 14:27
@Hotell Hotell changed the title perf: speed up tsc by not running on cypress files during .d.ts generation and type-checking perf: remove cypress test files from tsc Program for .d.ts generation and type-checking Apr 8, 2024
@Hotell Hotell changed the title perf: remove cypress test files from tsc Program for .d.ts generation and type-checking perf: exclude cypress test files from tsc Program for .d.ts generation and type-checking Apr 8, 2024
@Hotell Hotell merged commit 31be1ad into microsoft:master Apr 11, 2024
19 checks passed
@Hotell Hotell deleted the type-checking-perf/prerequirements/fix-cypress-tsc-slowdonws branch April 11, 2024 09:39
robertpenner pushed a commit to robertpenner/fluentui that referenced this pull request Apr 11, 2024
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.

5 participants