-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update esm-d-ts
to generate .d.cts
file / update dependencies
#19
Conversation
LGTM, thanks! |
Actually, one other to-do. Let's bump |
Will do this soon; usually unless the package being developed actually fails / requires that bump it wouldn't really be necessary. Sometimes it can be a pain when the basic dev tooling requires more recent versions when it comes to supporting the long tail. Even the latest Rollup / ESLint is |
It's healthy I think on our part to have a bump anyways since it allows us not to be tied to older language features. (It also of course lets us avoid testing older versions.) Actually, speaking of which, if you could also bump |
As far as |
Using |
This update refreshes / grabs the latest
esm-d-ts
which has a new--emitCTS
option to generate a duplicate output /.d.cts
file to reference inpackage.json
/require
export ensuring that downstream Typescript projects that are configured forCJS
work correctly.Updated all dependencies / fresh lock file. PNPM apparently has this issue where
pnpm up
does not update peer dependencies. In this case creating a fresh lock file from the latest dependency update was necessary to get the most recent of TypeDoc installed which is a peer dependency oftypedoc-pkg
.Updated export conditions in
package.json
to point to separate.d.cts
file for therequire
export condition.Regenerated docs (this accounts for the majority of files changed).
Update CI / dropping Node
16.x
for automatic testing asvitest
and possibly other dependencies require Node18+
. I added Node22.x
to the testing matrix and all is good.