From 72d8c4446feb829073f337410339288d268b880b Mon Sep 17 00:00:00 2001 From: Siddarth Kumar Date: Tue, 16 Jan 2024 23:30:42 +0530 Subject: [PATCH] chore: tiny `Makefile` fixes (#18521) Here we remove `yarn install` from `make component-test` and `make component-test-watch` commands. We also fix an extra `@` present ahead of `@scripts/check-metro-shadow-process.sh` inside `component-test-watch` command. --- Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 2cd0736391..208dba898d 100644 --- a/Makefile +++ b/Makefile @@ -386,9 +386,8 @@ component-test-watch: export COMPONENT_TEST := true component-test-watch: export BABEL_ENV := test component-test-watch: export JEST_USE_SILENT_REPORTER := false component-test-watch: ##@ Watch tests and re-run no changes to cljs files - @@scripts/check-metro-shadow-process.sh + @scripts/check-metro-shadow-process.sh rm -rf ./component-spec - yarn install nodemon --exec 'yarn shadow-cljs compile component-test && jest --config=test/jest/jest.config.js --testEnvironment node ' -e cljs component-test: export TARGET := clojure @@ -398,7 +397,6 @@ component-test: export JEST_USE_SILENT_REPORTER := false component-test: ##@test Run component tests once in NodeJS @scripts/check-metro-shadow-process.sh rm -rf ./component-spec - yarn install yarn shadow-cljs compile component-test && \ jest --clearCache && jest --config=test/jest/jest.config.js --testEnvironment node