search

jsr-io / jsr-npm

A cli tool to make installing packages form jsr.io in node easy
https://npmjs.com/package/jsr
MIT License
100 stars 12 forks source link

fix: Build `dist-esm` before publishing release on npm #64

Closed sebbo2002 closed 6 months ago

sebbo2002 commented 6 months ago

I am currently trying to use this module automatically. I have noticed that the dist-esm folder specified in package.json does not exist. I looked at the GitHub Action workflow file and saw that only npm publish is called here. Within the package.json, however, only tsc is called in the prepublishOnly script, not tsc -p tsconfig.esm.json, which is responsible for the ESM build. I have adapted the line so that npm run build is now executed (whch executes both builds).

I hope that this fixes the problem in the next release and would of course be delighted if this happes soon. Thank you.

CLAassistant commented 6 months ago

CLA assistant check
All committers have signed the CLA.