v3.0.1 introduces breaking change: no longer can be used in CJS

sindresorhus / vaca

Get a random ASCII cow

MIT License

102 stars 4 forks source link

Closed dylang closed 2 hours ago

dylang commented 3 hours ago

Hi again!

Your change for my previous ticket broke support for using vaca in CJS packages. I don't think that was intentional, as it wasn't a major bunmp.

Could we add "main": "./main.js" to the package.json?

Or an exports that supports both import and require?

sindresorhus commented 2 hours ago

It was never documented as being available as an API. It was just accidentally available because of the looseness of CJS.

sindresorhus commented 2 hours ago