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

Jackw/bundle size workflow #46

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Jackw/bundle size workflow #46

wants to merge 11 commits into from

Conversation

jackw
Copy link
Owner

@jackw jackw commented Oct 21, 2024

PR for adding bundle size action.

First we increase the bundle size by adding @mui/material to the project...

Copy link
Contributor

github-actions bot commented Oct 21, 2024

Bundle Size Changes

Hello! 👋 This comment was generated by a Github Action to help you and reviewers understand the impact of your PR on frontend bundle sizes.

Whenever this PR is updated, this comment will update to reflect the latest changes.

EntryPoint Size % diff
module 92.37 KB (+67.68 KB) +274.06%
Files Total bundle size % diff
10 420.15 KB (+67.68 KB) +19.20%
View detailed bundle information

Added

No assets were added

Removed

No assets were removed

Bigger

Name Size % Diff
module.js 92.37 KB (+67.68 KB) +274.06%

Smaller

No assets were smaller

View module information

Added

Name Size % Diff
@mui/material 129.08 KB (+129.08 KB) -
@mui/system 79.41 KB (+79.41 KB) -
@mui/utils 11.29 KB (+11.29 KB) -
react-transition-group 11.07 KB (+11.07 KB) -
@emotion/styled 8.5 KB (+8.5 KB) -
@emotion/serialize 6.19 KB (+6.19 KB) -
@emotion/is-prop-valid 4.41 KB (+4.41 KB) -
@mui/styled-engine 1.92 KB (+1.92 KB) -
@emotion/utils 1.72 KB (+1.72 KB) -
@emotion/hash 1.62 KB (+1.62 KB) -
@babel/runtime 1.22 KB (+1.22 KB) -
react 1.05 KB (+1.05 KB) -
@emotion/unitless 936 Bytes (+936 Bytes) -
@emotion/use-insertion-effect-with-fallbacks 472 Bytes (+472 Bytes) -
clsx 388 Bytes (+388 Bytes) -
@emotion/memoize 202 Bytes (+202 Bytes) -

Removed

No modules were removed

Bigger

Name Size % Diff
./components/TrafficLightPanel.tsx 7.3 KB (+1.12 KB) +18.16%

Smaller

No modules were smaller

@jackw jackw force-pushed the jackw/bundle-size-workflow branch 2 times, most recently from f2080d3 to 1c166d2 Compare November 8, 2024 07:08
@jackw
Copy link
Owner Author

jackw commented Nov 12, 2024

Now we're gonna trim some of the fat with some code splitting...

Copy link
Contributor

github-actions bot commented Nov 12, 2024

Bundle Size Changes

Hello! 👋 This comment was generated by a Github Action to help you and reviewers understand the impact of your PR on frontend bundle sizes.

Whenever this PR is updated, this comment will update to reflect the latest changes.

EntryPoint Size % diff
module 13.22 KB (-11.47 KB) -46.45%
Files Total bundle size % diff
10 → 15 356.71 KB (+4.24 KB) +1.20%
View detailed bundle information

Added

Name Size % Diff
904.js 5.3 KB (+5.3 KB) -
611.js 3.7 KB (+3.7 KB) -
943.js 3.57 KB (+3.57 KB) -
527.js 1.84 KB (+1.84 KB) -
95.js 1.31 KB (+1.31 KB) -

Removed

No assets were removed

Bigger

No assets were bigger

Smaller

Name Size % Diff
module.js 13.22 KB (-11.47 KB) -46.45%
View module information

Added

No modules were added

Removed

No modules were removed

Bigger

Name Size % Diff
./components/TrafficLightPanel.tsx 7.27 KB (+1.09 KB) +17.59%
./components/ThresholdsAssistant.tsx 3.2 KB (+8 Bytes) +0.24%
./components/TrafficLightDefault.tsx 4.89 KB (+8 Bytes) +0.16%
./components/TrafficLightRounded.tsx 4.76 KB (+8 Bytes) +0.16%
./components/TrafficLightSideLights.tsx 7.21 KB (+8 Bytes) +0.11%
./components/TrafficLightDynamic.tsx 2.65 KB (+8 Bytes) +0.30%

Smaller

No modules were smaller

Copy link
Contributor

github-actions bot commented Nov 12, 2024

Bundle Size Changes

Hello! 👋 This comment was generated by a Github Action to help you and reviewers understand the impact of your PR on frontend bundle sizes.

Great job! Bundle size increase of 0% is now below threshold of 5%.

Whenever this PR is updated, this comment will update to reflect the latest changes.

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.

1 participant