preact-render-to-string icon indicating copy to clipboard operation
preact-render-to-string copied to clipboard

Remove all default exports (major)

Open marvinhagemeister opened this issue 4 years ago • 1 comments

Mixing both default and named exports has no clearly defined behaviour and can easily lead to unexpected results. Some tools use a combination of default + __esModule property and others ignore named exports completely if both are found. Those issues are difficult to debug when multiple tools are used together.

This change makes some code redundant by not having to patch a hand crafted exports object for commonjs environments.

marvinhagemeister avatar Mar 10 '21 14:03 marvinhagemeister

Couldn't we provide only a default export? None of the other exported methods were widely used.

developit avatar Mar 15 '21 15:03 developit