sindresorhus / math-clamp

Clamp a number
MIT License
14 stars 3 forks source link

Move default condition last #15

Closed alebelcor closed 10 months ago

alebelcor commented 10 months ago

Hi,

There's a compilation error happening with Next.js when using math-clamp v2.2.0:

Module not found: Default condition should be last one

However, it doesn't happen with v2.1.0. It was most likely caused by #14 because it's the only commit contained in v2.2.0.

More specifically caused by the changes in package.json where a default exports entry was added and was not the last one, as the error indicates.

As reference, a similar issue was reported in firebase-js-sdk. The root cause was the same, a PR where package.json changed in a similar matter. The solution was to move the entry to be last as shown in this PR.

This PR attempts to fix the issue in the same way.

Feel free to let me know if I should make further changes.

Thanks!