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

npm ci does not run prepare script for Git modules (version 6.11.*) #310

Closed
darcyclarke opened this issue Nov 8, 2019 · 3 comments
Closed
Labels
Bug thing that needs fixing

Comments

@darcyclarke
Copy link
Contributor

  Original bug ticket: [https://npm.community/t/10076](https://npm.community/t/10076)
  Originally filed: 2019-09-16T18:40:45.299Z
@darcyclarke darcyclarke added Bug Bug thing that needs fixing and removed Community labels Nov 8, 2019
@daneah
Copy link

daneah commented Nov 17, 2019

Confirming this is still an issue in 6.12.0 as well.

@jjhbw
Copy link

jjhbw commented Mar 22, 2020

I have the exact same issue when using the node:12.13.0 Docker container (in CI), which runs 6.12.0. Same issue even after updating its npm version to 6.14.3 before running npm ci.

However, somehow, npm ci in 6.14.3 behaves as intended on my development machine (a mac)

@funkydev
Copy link

I have the same issue when using the node:12.13.0 on the docker container (in gitlab-runner CI) on both commands: npm install and npm ci.

I noticed that npm ignores files from .gitignore when there is no .npmignore file.
Creating an empty .npmignore file helps to solve problems locally (on a macOS system).

The strangest thing is that when I remove the node_modules directory and run the following command locally, it works.

$ docker run --rm --name node-test -v "$PWD":/usr/src/app -w /usr/src/app node:12.13.0 \
             sh -c 'npm cache clean --force && npm ci'

I have problems only when trying to run the following command on the CI runner (that uses node:12.13.0 docker image).

@darcyclarke darcyclarke modified the milestone: OSS - Sprint 17 Oct 5, 2020
dubiousdavid added a commit to dubiousdavid/node-resque that referenced this issue Jan 25, 2021
jacksongross added a commit to streemau/vue-draggable-resizable that referenced this issue Dec 16, 2021
Signed-off-by: Jackson Gross <jackson.gross@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug thing that needs fixing
Projects
None yet
Development

No branches or pull requests

4 participants