-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[BUG] When installing a git dependency, prepare script not running #1287
Comments
I think this is another manifestation of what happens when there is an From docs (I don't understand the reasoning behind this choice):
While you do seem to have a From docs, emphasis mine:
Note it mentions "files" and not directories. Fixes to try:
|
I’ll try those. Thank you. |
Add an empty
|
@cinderblock didn't seem either of those helped. |
Oh well, worth a shot. I've seen that issue happen too many times. I'm reading the logs and seeing a couple errors that stand out. I'm not sure what they mean exactly but are worth investigating I think:
You might also try turning on verbose npm debugging in a https://docs.npmjs.com/using-npm/config#loglevel Alternatively, checking for any |
npm If your bug is preproducible on If your issue was a feature request, please consider opening a new RRFC or RFC. If your issue was a question or other idea that was not CLI-specific, consider opening a discussion on our feedback repo Closing: This is an automated message. |
...for the issue: npm/cli#1287
Not sure if this will work; taken from npm/cli#1287 (comment)
Not sure if this will work; taken from npm/cli#1287 (comment)
Added empty .npmigore for npm/cli#1287
For anybody else running into this issue and finding this thread: forget about using
This does mean you're essentially pinned to the tarball's version, since NPM does not allow using a #semver range with tarball URLs... c'est la vie. |
What / Why
The
prepare
script isn't being run on install.When
Where
6.14.5
in Docker container.How
Current Behavior
dist
directory is created.Steps to Reproduce
This should end up with
dist
being shown in thels
.mkdir x && cd x && npm init -y
npm i github:omgimalexis/execution-time -ddd
ls node_modules/execution-time/
This should end up with
dist
missing in thels
.docker run -it mhart/alpine-node:12 sh
apk add --no-cache git
mkdir x && cd x && npm init -y
npm i github:omgimalexis/execution-time -ddd
ls node_modules/execution-time/
Expected Behavior
dist
directory is created.Who
References
Possibly related to #1229 or #1214
Both are running npm
6.14.5
and are on node12
.Container silly logs
Local silly logs
The text was updated successfully, but these errors were encountered: