From 3cc95c474f5964ac4bb77dd9cc0d58788f52f5da Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fran=C3=A7ois=20Freitag?= Date: Sat, 20 Feb 2021 19:56:02 +0100 Subject: [PATCH] Use npm ci to install front-end dependencies in dev MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The ci command is meant for any situation where you want to make sure you're doing a clean install of your dependencies. It is significantly faster than npm install. If a node_modules is already present, it will be automatically removed before npm ci begins its install. Matches the GitHub actions to build the package. The clean-dev rule is now unused and can be removed. https://docs.npmjs.com/cli/v7/commands/npm-ci Signed-off-by: François Freitag --- Makefile | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/Makefile b/Makefile index 0fa4456ce..7cc29e20d 100644 --- a/Makefile +++ b/Makefile @@ -4,10 +4,10 @@ all: dev-setup lint build-js-production test test-php # Dev env management -dev-setup: clean clean-dev npm-init +dev-setup: clean npm-init npm-init: - npm install + npm ci composer-init: composer install --prefer-dist @@ -62,9 +62,6 @@ stylelint-fix: clean: rm -rf js -clean-dev: - rm -rf node_modules - # Builds the source package for the app store, ignores php and js tests appstore: krankerl package