-
Notifications
You must be signed in to change notification settings - Fork 56
Update @manypkg/*
packages to be ESM only
#242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🦋 Changeset detectedLatest commit: 6c863dd The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
@manypkg/*
packages to be ESM only, publish @manypkg/*
packages to be ESM only
* Fix TS issues * fix knip * fix tests
Co-authored-by: Mateusz Burzyński <[email protected]>
It was mentioned in the e18e discord that manypkg might be interested in moving it's packages to ESM-only.
This is a draft PR that:
type: "module"
to all@manypkg/
packagesI really tried to avoid exploding scope, but along the way I:
node-version
file, updating a previously implicit 18 -> to explicit 22 (might revert this bump, it has wider implications)Putting a draft PR up before I sleep, I think just need to fix the manypkg/cli loader error ⬇️