Skip to content

Add side effects to package.json#271

Open
denisx wants to merge 1 commit into
vueuse:mainfrom
denisx:patch-1
Open

Add side effects to package.json#271
denisx wants to merge 1 commit into
vueuse:mainfrom
denisx:patch-1

Conversation

@denisx

@denisx denisx commented Dec 6, 2024

Copy link
Copy Markdown

Add side effects to package.json, help bundlers cut unused code via treeshake. Good with esm, modules, named imports.

@antfu

antfu commented Dec 10, 2024

Copy link
Copy Markdown
Member

This library does have side-effects 👀

And what's your usage of this library? It will soon be deprecated

@denisx

denisx commented Dec 10, 2024

Copy link
Copy Markdown
Author

"side effects = false " show it. if we have side effects, we make a list of imports

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.

2 participants