Add sideeffects: false to package.json

[Travis DePrato]

See title.

Allows for deadcode elimination when not the module is imported but not actually used.

This comes up for me when using NextJS where one frequently writes server side code as a getServerSideProps function. I want to import Node's crypto module but don't need it for the browser -- but webpack is unable to remove the import even though Next does remove the getServerSideProps function).

The workaround is to add

module.exports = {
  node: {
    crypto: "empty"
  }
}

to the webpack config.

[Mitchell Heddles]

@travigd Can you use a dynamic import instead?

export async function getServerSideProps() {
  const crypto = await import('crypto');
}

[Travis DePrato]

Yes, but this should still be fixed. :)

I'm currently just using the webpack workaround described above.

[rawr51919]

Could probably integrate this into PR #200 tbh. Will here shortly

[Jordan Harband]

Probably best not to conflate multiple things in the same PR.

[rawr51919]

Would it be best to rather make a separate PR after #200 is pushed to have this fixed?

[Jordan Harband]

I'd think so.

[rawr51919]

PR #201 will fix this issue

[Jordan Harband]

Fixed in b638139

[rawr51919]

Fixed in b638139

glad you got around to it thanks :)