-
Notifications
You must be signed in to change notification settings - Fork 3
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
Do not append, instead clobber. #1
Comments
Yep you're probably right. Could allow clobbering via option. Not sure what the default should be. I haven't used this module in awhile. Initially I was wanting to create a module that would track installed modules. A sort of analytics module. |
I found an incredibly important use case for this (or hooks in general). We created a far superior postinstall for the purpose of architecture specific compilation, but it could be used for any build step in postinstall. It's called
Here's what the package.json would look like: {
"name": "testtop",
"version": "1.0.0",
"scripts": {
"install": "esybuild"
},
"esy": {
"build": ["your commands here"]
},
"dependencies": {
"esy": "*"
}
} Here's where I use my fork of this library to do that. What it does is, it attaches a hook to ensure that only the top level package's post install runs, which kicks of the parallel builds from the top down in a system we control, not Notice how it just looks like you're running |
I found it much more useful in my fork of npm-hooks, to clobber instead of append the file. Do you agree this should be a supported mode?
The text was updated successfully, but these errors were encountered: