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

Allow treeshaking with @tabler/icons-solidjs #1221

Merged
merged 1 commit into from
Sep 26, 2024
Merged

Conversation

inetol
Copy link
Contributor

@inetol inetol commented Sep 13, 2024

In my case, I have not seen any issues setting the sideEffects flag globally. This should fix all bundling size issues

Closes #1215
Closes #1203

Copy link

vercel bot commented Sep 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
tabler-icons ✅ Ready (Inspect) Visit Preview Sep 13, 2024 9:23am

@codecalm
Copy link
Member

thank you for your help! ❤️

@codecalm codecalm merged commit dca84ef into tabler:main Sep 26, 2024
2 of 3 checks passed
@inetol inetol deleted the bundlefix branch September 27, 2024 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@tabler/icons-solidjs 3.x.x bundle size issue Building with Vite and SolidJS includes all icons
2 participants