Skip to content

Commit cdb7476

Browse files
committed
fix(makefile): set help as default target instead of parameter error
- Add .DEFAULT_GOAL := help to make 'make' show help by default - Previously 'make' without arguments showed parameter validation error - Now provides better UX by showing comprehensive help output - Preserves parameter validation for infrastructure commands that need them - Fixes common user frustration when exploring available commands Improves developer experience for Issue #28 staging deployment testing.
1 parent 38c9e3d commit cdb7476

File tree

1 file changed

+8
-12
lines changed

1 file changed

+8
-12
lines changed

Makefile

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -9,18 +9,14 @@
99
.PHONY: vm-ssh vm-console vm-gui-console vm-clean-ssh vm-prepare-ssh vm-status
1010
.PHONY: dev-setup dev-deploy dev-test dev-clean
1111

12-
# Default variables
13-
VM_NAME ?= torrust-tracker-demo
14-
# Defaults for quick development workflows only
15-
DEV_ENVIRONMENT_TYPE ?= development
16-
DEV_ENVIRONMENT_FILE ?= development-libvirt
17-
DEV_PROVIDER ?= libvirt
18-
TERRAFORM_DIR = infrastructure/terraform
19-
INFRA_TESTS_DIR = infrastructure/tests
20-
TESTS_DIR = tests
21-
SCRIPTS_DIR = infrastructure/scripts
22-
23-
# Parameter validation for infrastructure commands
12+
# Default environment variables
13+
ENVIRONMENT_TYPE ?= development
14+
ENVIRONMENT_FILE ?= development-libvirt
15+
16+
# Default target - show help when no target specified
17+
.DEFAULT_GOAL := help
18+
19+
# Parameter validation target
2420
check-infra-params:
2521
@if [ -z "$(ENVIRONMENT_TYPE)" ]; then \
2622
echo "❌ Error: ENVIRONMENT_TYPE not specified"; \

0 commit comments

Comments
 (0)