- Notifications
You must be signed in to change notification settings - Fork317
build: Flagged module as sideEffects false to enhance tree shaking#355
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
Conversation
alanhe421 commentedSep 28, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Maybe you can help review the PR? and merge it. thanks. |
AlixWang commentedOct 17, 2024
this repo seems inactive, so may be we should find a new one |
ok. I created a branch for maintenance. It has also been published to npm, and can be used directly if needed. installnpm i @stacker/redux-logger links |
The current v4 version already supports the ES module, so in order for webpack packaging to work properly for treeshaking, you should mark the sideEffects here.