Skip to content
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

Added publish npm script/hook to ensure runtime gets built when referenced as a git repo #239

Merged
merged 2 commits into from
Oct 14, 2023

Conversation

axmmisaka
Copy link
Collaborator

@axmmisaka axmmisaka commented Oct 13, 2023

Behavior changed because prepublish was changed to prepublishonly due to prettier suggestion (?), but it's not consistent.
idk why it doesn't work with npm in the past. npm should run prepublish as well.

Now we use prepare, which will run before publish and install.

squash and merge plz

@axmmisaka axmmisaka changed the title install npm script/hook added to ensure runtime gets built when referenced as a git repo publish npm script/hook added to ensure runtime gets built when referenced as a git repo Oct 14, 2023
@lhstrh lhstrh changed the title publish npm script/hook added to ensure runtime gets built when referenced as a git repo Added publish npm script/hook to ensure runtime gets built when referenced as a git repo Oct 14, 2023
@lhstrh lhstrh self-requested a review October 14, 2023 00:33
@lhstrh lhstrh merged commit 9798f65 into master Oct 14, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants