-
-
Notifications
You must be signed in to change notification settings - Fork 968
Description
Hi! Apologies if there was already an issue for this, I didn't find any.
Just found out Got only provides an ESM version, but no CJS version, which makes it incompatible with CommonJS projects.
After digging through this repo I found a Gist from sindersorhus confirming that the package did indeed remove support for CommonJS.
A Gist is nice, but a lack of support like this should IMHO be mentioned clearly in the README as it can be frustrating to users (especially newbies).
Something like this before the 'Install' section would be nice:
Warning: This package no longer provides a CJS module. If your project uses CommonJS, you'll have to convert to ESM, or use the dynamic
import()
function. Please don't open issues for questions regarding CommonJS / ESM.
WDYT?