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

Add a prepublishOnly script for release #157

Closed
wants to merge 1 commit into from

Conversation

jonkoops
Copy link
Contributor

Adds a prepublishOnly script, which will run before the package is published through npm publish. This script will run the build and tests, so that a broken package is not shipped by accident.

@jonkoops jonkoops requested a review from a team as a code owner July 26, 2023 15:55
@frederikprijck
Copy link
Member

frederikprijck commented Jul 26, 2023

Thanks, we always run npm run build already on our infrastructure prior to publishing automatically, this would unneccesarily add an additional build.

Leaving it open as i want to see if it would make sense, but im afraird we might not pick up this change to avoid having to change our release set up.

@jonkoops
Copy link
Contributor Author

Just making sure, I will go ahead and close this one then 👍

@jonkoops jonkoops closed this Jul 26, 2023
@jonkoops jonkoops deleted the prepublish-script branch July 26, 2023 16:40
@jonkoops
Copy link
Contributor Author

Looks like I was a bit too early and closed this out before your edit, feel free to re-open if it makes sense.

@frederikprijck
Copy link
Member

We needed this, sorry about that. I added it in #167 using prepack.

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