Skip to content

🐛 run generate step as postinstall script#407

Merged
carlosthe19916 merged 3 commits intoguacsec:mainfrom
kahboom:generate-postinstall-fix
Mar 22, 2025
Merged

🐛 run generate step as postinstall script#407
carlosthe19916 merged 3 commits intoguacsec:mainfrom
kahboom:generate-postinstall-fix

Conversation

@kahboom
Copy link
Copy Markdown
Collaborator

@kahboom kahboom commented Mar 21, 2025

This guarantees that the generate script is run for anyone prior to using the package (post-install or after npm install). See here for more information: https://docs.npmjs.com/cli/v8/using-npm/scripts#prepare-and-prepublish

@carlosthe19916 carlosthe19916 changed the title fix: run generate step as postinstall script 🐛 run generate step as postinstall script Mar 21, 2025
Signed-off-by: Carlos Feria <2582866+carlosthe19916@users.noreply.github.com>
@carlosthe19916
Copy link
Copy Markdown
Collaborator

LGTM! I just resolved the conflicts in package.json

@carlosthe19916 carlosthe19916 merged commit b165171 into guacsec:main Mar 22, 2025
8 checks passed
@kahboom kahboom deleted the generate-postinstall-fix branch April 4, 2025 15:25
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