-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
chore: 🔧 update postinstall script #33
Conversation
CHANGELOG.md
Outdated
## v2.2.1 | ||
_2023-10-10_ | ||
* [#33](https://github.com/perry-mitchell/ulidx/pull/33): Use prepare instead of postinstall for git hooks | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need for version bump, that's part of my release process :)
package.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "ulidx", | |||
"version": "2.2.0", | |||
"version": "2.2.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here..
Thanks for this! I'll get it released in a moment. |
Released under 2.2.1. Thanks for spotting and sorry for the inconvenience! |
Hi. Renovate updated ulidx to 2.2.0 and our pipeline failed. I believed the issue is using the
postinstall
script to setup git hooks instead ofprepare
, which runs afternpm install
.Because
postinstall
will run when a dependency installs their packages, whileprepare
will only ever run locally, not for dependencies of ulidx.Might close #32