search

cefn / zx-tsnode-repro

MIT License
0 stars 1 forks source link

Adopt @cspotcode/zx #3

Open cefn opened 2 years ago

cefn commented 2 years ago

Working configuration of ts-node with @cspotcode/zx that fixes https://github.com/cefn/zx-tsnode-repro/issues/1 . To prove it, run npm install && npm run test.

This is a simpler strategy to get a working zx with ts-node than trying to keep up with the breaking changes of zx@6 as embodied in the alternative PR.

However, it effectively pins zx to version 5, as @cspotcode's version is not aligned with the latest zx release.

cspotcode commented 2 years ago

I'm happy to publish a release of @cspotcode/zx that merged in the latest changes from v6

cspotcode commented 2 years ago

I've published @cspotcode/zx 6.1.0 which includes all the latest changes from zx. Do you have a minute to try it out and double-check that it works for you?

cefn commented 2 years ago

Super! To do this justice I should try it against our real tooling scripts when I'm in work on Monday again.

It's tempting to depend on a version proven to work alongside ts-node but I suppose we should be aware that we would 'silently' be detached from the ongoing features and security patches of zx although this is much less significant for our tooling scripts than for production code.

cspotcode commented 2 years ago

Definitely. For future readers: I created this other example which shows using ts-node's native ESM support to use zx instead of @cspotcode/zx https://github.com/cefn/zx-tsnode-repro/pull/4