github / webpack-bundlesize-compare-action

A github action that outputs bundlesize comparison tables for pull requests
MIT License
78 stars 19 forks source link

streamline changest viewing for larger diffs #297

Closed mattcosta7 closed 1 year ago

mattcosta7 commented 1 year ago

We basically output the same thing twice, and on large diffs we'll go well over the comment size limit for an issue.

streamlining this by minimize the necessary output, removing the 'detailed' view, in favor of updating the changeset to contain added info about the file sizes

while here I also updated some dependencies

fixes https://github.com/github/webpack-bundlesize-compare-action/pull/296 fixes https://github.com/github/webpack-bundlesize-compare-action/pull/294 fixes https://github.com/github/webpack-bundlesize-compare-action/pull/293

github-actions[bot] commented 1 year ago

Bundle Stats-With chunk and module information

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files count Type Total bundle size % Changed
2 bundled 1.98 MB 0%
**Changeset** File | Δ | Size ---- | - | ---- `src/client/this/file/has/a/long/name/so-that-it-will-hopefully-wrap.tsx` | 🆕 +2.72 KB | 0 B -> 2.72 KB `src/client/this-file-was-added.ts` | 🆕 +1.46 KB | 0 B -> 1.46 KB `src/client/this-is-another-file-with-a-long-name-to-test-the-table-sizing.tsx` | 🆕 +1.04 KB | 0 B -> 1.04 KB `src/client/helpers/this-file-is-much-larger.ts` | 📈 +11.66 KB (+865.77%) | 1.35 KB -> 13.01 KB `src/client/this-file-grew-larger.tsx` | 📈 +200 B (+35.59%) | 562 B -> 762 B `src/client/helpers/this-file-grew-smaller.ts` | 📉 -200 B (-7.94%) | 2.46 KB -> 2.26 KB `src/client/this-file-is-much-smaller.tsx` | 📉 -11.66 KB (-99.92%) | 11.67 KB -> 10 B `src/client/routes.tsx` | 🔥 -2.72 KB (-100%) | 2.72 KB -> 0 B `src/client/this-file-will-be-deleted.ts` | 🔥 -1.46 KB (-100%) | 1.46 KB -> 0 B `src/client/render-memex.tsx` | 🔥 -1.04 KB (-100%) | 1.04 KB -> 0 B
View detailed bundle breakdown
**Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | Type | File Size | % Changed ----- | ---- | --------- | --------- app.bundle.js | bundled | 1.98 MB | 0% manifest.json | bundled | 91 B | 0%
github-actions[bot] commented 1 year ago

Bundle Stats

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files count Type Total bundle size % Changed
7 -> 2 bundled
gzip
1.34 MB -> 1.29 MB (-53.65 KB)
386.44 KB -> N/A
-3.91%
View detailed bundle breakdown
**Added** No assets were added **Removed** Asset | Type | File Size | % Changed ----- | ---- | --------- | --------- 296.chunk.js | bundled
gzip | 124.57 KB -> 0 B (-124.57 KB)
35.05 KB -> 0 B | -100% 288.chunk.js | bundled
gzip | 57.24 KB -> 0 B (-57.24 KB)
16.33 KB -> 0 B | -100% 920.chunk.js | bundled
gzip | 54.98 KB -> 0 B (-54.98 KB)
17.08 KB -> 0 B | -100% 912.chunk.js | bundled
gzip | 44.37 KB -> 0 B (-44.37 KB)
14.31 KB -> 0 B | -100% 699.chunk.js | bundled
gzip | 26.39 KB -> 0 B (-26.39 KB)
6.14 KB -> 0 B | -100% **Bigger** Asset | Type | File Size | % Changed ----- | ---- | --------- | --------- app.bundle.js | bundled
gzip | 1.04 MB -> 1.29 MB (+254.35 KB)
297.38 KB -> N/A | +23.91% **Smaller** Asset | Type | File Size | % Changed ----- | ---- | --------- | --------- manifest.json | bundled
gzip | 551 B -> 91 B (-460 B)
151 B -> N/A | -83.48% **Unchanged** No assets were unchanged
github-actions[bot] commented 1 year ago

Bundle Stats-Flipped around

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files count Type Total bundle size % Changed
2 -> 7 bundled
gzip
1.29 MB -> 1.34 MB (+53.65 KB)
N/A -> 386.44 KB
+4.07%
View detailed bundle breakdown
**Added** Asset | Type | File Size | % Changed ----- | ---- | --------- | --------- 296.chunk.js | bundled
gzip | 0 B -> 124.57 KB (+124.57 KB)
0 B -> 35.05 KB | - 288.chunk.js | bundled
gzip | 0 B -> 57.24 KB (+57.24 KB)
0 B -> 16.33 KB | - 920.chunk.js | bundled
gzip | 0 B -> 54.98 KB (+54.98 KB)
0 B -> 17.08 KB | - 912.chunk.js | bundled
gzip | 0 B -> 44.37 KB (+44.37 KB)
0 B -> 14.31 KB | - 699.chunk.js | bundled
gzip | 0 B -> 26.39 KB (+26.39 KB)
0 B -> 6.14 KB | - **Removed** No assets were removed **Bigger** Asset | Type | File Size | % Changed ----- | ---- | --------- | --------- manifest.json | bundled
gzip | 91 B -> 551 B (+460 B)
N/A -> 151 B | +505.49% **Smaller** Asset | Type | File Size | % Changed ----- | ---- | --------- | --------- app.bundle.js | bundled
gzip | 1.29 MB -> 1.04 MB (-254.35 KB)
N/A -> 297.38 KB | -19.29% **Unchanged** No assets were unchanged
github-actions[bot] commented 1 year ago

Bundle Stats-Non existent

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files count Type Total bundle size % Changed
0 bundled 0 B 0%
View detailed bundle breakdown
**Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** No assets were unchanged
github-actions[bot] commented 1 year ago

Bundle Stats-No changes

Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.

As this PR is updated, I'll keep you updated on how the bundle size is impacted.

Total

Files count Type Total bundle size % Changed
7 bundled
gzip
1.34 MB
386.44 KB
0%
View detailed bundle breakdown
**Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | Type | File Size | % Changed ----- | ---- | --------- | --------- app.bundle.js | bundled
gzip | 1.04 MB
297.38 KB | 0% 296.chunk.js | bundled
gzip | 124.57 KB
35.05 KB | 0% 288.chunk.js | bundled
gzip | 57.24 KB
16.33 KB | 0% 920.chunk.js | bundled
gzip | 54.98 KB
17.08 KB | 0% 912.chunk.js | bundled
gzip | 44.37 KB
14.31 KB | 0% 699.chunk.js | bundled
gzip | 26.39 KB
6.14 KB | 0% manifest.json | bundled
gzip | 551 B
151 B | 0%
mattcosta7 commented 1 year ago

Once this lands, we should version and update internal consumers