-
Notifications
You must be signed in to change notification settings - Fork 1
Add prepare script to force npm publish
to include built JS
#58
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
base: main
Are you sure you want to change the base?
Conversation
I think this fix won't be sufficient. If you look at the npm package for
Thus Running build script will fail:
|
The tsconfig doesn't have to be included since the intention is to have it include a precompiled build in the released NPM package. |
right you're correct I thought of post install. I wonder if you plan to fix the issue and publish fixed version which includes compiled assets @dhkatz? |
The latest release (
1.0.1
) at the time of writing, did not have adist
, so importing the library didn't work.This fixes that so the build is always run during publish. (
prepare
is an npm hook)✨