Remove duplicate dependencies from the 'test' and 'dev' targets

elifesciences · Jan 9, 2025 · 105ab27 · 105ab27
1 parent 7950424
commit 105ab27
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/Makefile b/Makefile
@@ -1,17 +1,19 @@
 .PHONY: dev
-dev: config.php
-	docker compose up --wait
+dev: bring-up-all-services
 	docker compose logs --follow
 
+.PHONY: bring-up-all-services
+bring-up-all-services: config.php
+	docker compose up --wait
+
 .PHONY: check
 check: test
 	docker compose run --rm --no-deps app vendor/bin/phpcs --standard=phpcs.xml.dist --warning-severity=0 -p src/ tests/ web/
 	docker compose run --rm --no-deps app vendor/bin/composer-dependency-analyser
 	docker compose run --rm --no-deps app vendor/bin/phpstan analyse
 
 .PHONY: test
-test: config.php
-	docker compose up --wait
+test: bring-up-all-services
 	docker compose exec app vendor/bin/phpunit
 
 .PHONY: stop