Skip to content

TeachingPopover: Automated Tests#30836

Merged
Mitch-At-Work merged 3 commits into
microsoft:masterfrom
Mitch-At-Work:user/mifraser/teaching-popover-funcs
Mar 25, 2024
Merged

TeachingPopover: Automated Tests#30836
Mitch-At-Work merged 3 commits into
microsoft:masterfrom
Mitch-At-Work:user/mifraser/teaching-popover-funcs

Conversation

@Mitch-At-Work

Copy link
Copy Markdown
Contributor

Adds the automated checks for both bundle size and performance to TeachingPopover

@Mitch-At-Work Mitch-At-Work requested review from a team as code owners March 20, 2024 18:05
@fabricteam

fabricteam commented Mar 20, 2024

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against 0a593445b3809289313bc6d49dedd18f33a850e4

@codesandbox-ci

codesandbox-ci Bot commented Mar 20, 2024

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.

@tudorpopams tudorpopams requested a review from Hotell March 21, 2024 13:07
@Hotell

Hotell commented Mar 22, 2024

Copy link
Copy Markdown
Contributor

I guess you did this because it's part of our Convergence epic "steps" ?

This "perf" approach is outdated and provides no value for us as it uses flamegril which doesn't work in browsers that we support.

I'd suggest to close this and not merge or just remove those "perf test" and land bundle-size setup

@Mitch-At-Work

Copy link
Copy Markdown
Contributor Author

Updated to only enable bundle size

@Mitch-At-Work Mitch-At-Work merged commit 10e6d9d into microsoft:master Mar 25, 2024
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.

4 participants