Export uuid as default as well #465
Conversation
This allows people to do: ```js import uuid from 'uuid'; uuid.v4(); ``` It also [helps with this Jest issue](https://stackoverflow.com/a/62210156/938236), but I think allowing for the above on itself is merit enough for this PR.
|
@franciscop thank you for the suggestion. We deliberately removed the default export with the Could you elaborate on why adding a default export changes anything for jest in particular? Isn't this more a problem of Jest not yet supporting I think we are tracking this issue in #451. |
|
As noted in stackoverflow I think adding a default export to the esm build (while the CommonJS package does not have a default export) just for the sake of working around a current flaw in jest is not justified. I think it even has the potential of doing more harm than good because it will make testing |
|
@ctavan I think it's useful for people to be able to use the library either way, but if it was purposefully removed I'll close this PR. On the jest side, it's only until they support "Package Exports", so it only fixes a temporary bug. I explained it in detail on StackOverflow. |
|
OK. I prefer to solve this on the |
This allows people to do:
It also helps with this Jest issue, but I think allowing for the above on itself is merit enough for this PR.