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

Replace size-limit action with own metrics-report action #433

Merged
merged 42 commits into from
Jun 23, 2024

Conversation

dcastil
Copy link
Owner

@dcastil dcastil commented Jun 22, 2024

This is to improve the size reporting and in preparation for #253 which I might to some time later.

@github-actions github-actions bot added the other Is other label Jun 22, 2024
Copy link

github-actions bot commented Jun 22, 2024

size-limit report 📦 (outdated content)

Path Size
dist/bundle-mjs.mjs 6.06 KB (-0.28% 🔽)
dist/bundle-cjs.js 6.18 KB (+0.26% 🔺)
dist/es5/bundle-mjs.mjs 6.32 KB (+0.33% 🔺)
dist/es5/bundle-cjs.js 6.43 KB (+0.48% 🔺)

@dcastil
Copy link
Owner Author

dcastil commented Jun 22, 2024

Test test

We need to do that because the metrics-report-action checks out a different branch and installs node modules there which would break saving the cache in the post step
getting some weird warnings
Copy link

github-actions bot commented Jun 22, 2024

Metrics report

At head commit 2b01a89 and base commit b756669 at 2024-06-23T16:34:38.571Z

Size

Export Size original Size minified Size minified and Brotli compressed
tailwind-merge/es5 esm 74.72 kB -0.1% 🟢 21.70 kB -0.1% 🟢 6.00 kB -0.2% 🟢
 › createTailwindMerge 14.02 kB -0.4% 🟢 4.17 kB -0.3% 🟢 1.53 kB +0.1% 🔴
 › extendTailwindMerge 74.04 kB -0.1% 🟢 21.28 kB -0.1% 🟢 5.87 kB +0.4% 🔴
 › twMerge 72.10 kB -0.1% 🟢 20.65 kB -0.1% 🟢 5.66 kB +0.1% 🔴
tailwind-merge/es5 cjs 74.98 kB -0.1% 🟢 24.79 kB 0.0% 🟢 6.31 kB 0.0% 🟢
All size metrics
Export Size original Size minified Size minified and Brotli compressed
tailwind-merge esm 71.96 kB 0% 20.75 kB 0% 5.80 kB 0%
 › createTailwindMerge 12.44 kB 0% 3.65 kB 0% 1.37 kB 0%
 › extendTailwindMerge 71.47 kB 0% 20.32 kB 0% 5.63 kB 0%
 › fromTheme 0.15 kB 0% 0.08 kB 0% 0.08 kB 0%
 › getDefaultConfig 57.49 kB 0% 16.20 kB 0% 4.17 kB 0%
 › mergeConfigs 1.32 kB 0% 0.40 kB 0% 0.20 kB 0%
 › twJoin 1.06 kB 0% 0.26 kB 0% 0.16 kB 0%
 › twMerge 69.96 kB 0% 19.86 kB 0% 5.46 kB 0%
 › validators 3.21 kB 0% 1.54 kB 0% 0.66 kB 0%
tailwind-merge cjs 72.22 kB 0% 23.81 kB 0% 6.11 kB 0%
tailwind-merge/es5 esm 74.72 kB -0.1% 🟢 21.70 kB -0.1% 🟢 6.00 kB -0.2% 🟢
 › createTailwindMerge 14.02 kB -0.4% 🟢 4.17 kB -0.3% 🟢 1.53 kB +0.1% 🔴
 › extendTailwindMerge 74.04 kB -0.1% 🟢 21.28 kB -0.1% 🟢 5.87 kB +0.4% 🔴
 › fromTheme 0.19 kB 0% 0.10 kB 0% 0.09 kB 0%
 › getDefaultConfig 58.06 kB 0% 16.46 kB 0% 4.18 kB 0%
 › mergeConfigs 1.52 kB 0% 0.43 kB 0% 0.22 kB 0%
 › twJoin 1.06 kB 0% 0.25 kB 0% 0.15 kB 0%
 › twMerge 72.10 kB -0.1% 🟢 20.65 kB -0.1% 🟢 5.66 kB +0.1% 🔴
 › validators 3.37 kB 0% 1.53 kB 0% 0.66 kB 0%
tailwind-merge/es5 cjs 74.98 kB -0.1% 🟢 24.79 kB 0.0% 🟢 6.31 kB 0.0% 🟢

Repository owner deleted a comment from github-actions bot Jun 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
context-v2 Related to tailwind-merge v2 other Is other
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant