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

compile to es6 and set sideEffects: false to enable tree shaking #970

Merged
merged 1 commit into from
Feb 2, 2021

Conversation

andnorda
Copy link
Collaborator

@andnorda andnorda commented Feb 1, 2021

Hva er nedsiden ved å distribuere pakken som es6 framfor CommonJS?

Jeg fikk ikke til å tree shake icons med create-react-app når den var CommonJS, men i es6 funka det...

@andnorda andnorda requested a review from KenAJoh February 1, 2021 17:20
@andnorda andnorda requested a review from a team as a code owner February 1, 2021 17:20
@KenAJoh
Copy link
Collaborator

KenAJoh commented Feb 2, 2021

Hva er nedsiden ved å distribuere pakken som es6 framfor CommonJS?

Eneste jeg kan se for meg er at server-side rendering vil bli påvirket på en eller annen måte, men tviler på at det fortsatt er et problem i 2021.

Hva testet du tree shaking på? CRA har som jeg har skjønt det ganske kraftig tree shaking bygd inn, mens webpack i seg selv er ganske manglende. Hvis tree shaking fungerer for webpack har det vært flott.

@andnorda
Copy link
Collaborator Author

andnorda commented Feb 2, 2021

Har testa med webpack@5.19 nå, og det funker fett.

Hva tenker du kan skape krøll i server-side-redrede apper?

Copy link
Collaborator

@KenAJoh KenAJoh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@KenAJoh
Copy link
Collaborator

KenAJoh commented Feb 2, 2021

Siden Commonjs er språket nettlesere bruker kunne man slippe babel for transpilering og da sende koden direkte med SSR slik jeg har skjønt det. Men dette var lenge siden da og antar at transpilering er default i alle løsninger nå.

@KenAJoh KenAJoh merged commit 2f4f652 into master Feb 2, 2021
@KenAJoh KenAJoh deleted the tree-shakable-icons branch February 2, 2021 12:01
@andnorda
Copy link
Collaborator Author

andnorda commented Feb 2, 2021

Ja, jeg tror det er en trygg antagelse at alle team ruller sin egen .js-ressurs.

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.

None yet

2 participants