Merge pull request #1170 from nextcloud/fix/npm-ci

Usage of npm ci instead of npm install for automation scripts
nextcloud · Aug 23, 2022 · e90c1ed · e90c1ed
2 parents e62cdd4 + 6623bae
commit e90c1ed
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 4 deletions.
diff --git a/.github/actions/run-tests/tests/entrypoints/test.sh b/.github/actions/run-tests/tests/entrypoints/test.sh
@@ -76,7 +76,7 @@ fi
 
 if [ $BUILD_NPM = 'y' ]; then
 	echo 'Installing NPM packages'
-	npm install
+	npm ci
 
 	echo 'Building JS folder'
 	npm run build

diff --git a/.github/workflows/deploy-appstore.yml b/.github/workflows/deploy-appstore.yml
@@ -51,7 +51,7 @@ jobs:
                     BOT_TOKEN: ${{ secrets.COOKBOOK_BOT_TOKEN }}
 
             -   name: Install the NPM packages
-                run: npm install
+                run: npm ci
 
             -   name: Build NPM
                 run: npm run build

diff --git a/.github/workflows/pull-checks.yml b/.github/workflows/pull-checks.yml
@@ -131,7 +131,7 @@ jobs:
                         ${{ runner.os }}-node-
 
             -   name: Install NPM modules
-                run: npm install
+                run: npm ci
 
             -   name: Perform the test
                 run: |

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -421,7 +421,7 @@ jobs:
         -   name: Install the NPM dependencies
             shell: bash
             run: >-
-                npm install
+                npm ci
         -   name: Install the dependencies for composer
             shell: bash
             run: >-

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -53,6 +53,8 @@
   [#1159](https://github.com/nextcloud/cookbook/pull/1159) @christianlupus
 - Prepare the generation of pre-releases
   [#1169](https://github.com/nextcloud/cookbook/pull/1169) @christianlupus
+- Corrected usage of npm ci
+  [#1170](https://github.com/nextcloud/cookbook/pull/1170) @christianlupus
 
 
 ## 0.9.13 - 2022-07-02