refactor(Makefile): move .PHONY before each target

Move each .PHONY declaration to immediately precede its corresponding
target for better readability and maintainability.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>

Author: dannywillems

Makefile

@@ -62,10 +62,13 @@ WORKSPACE_FEATURES = \
 	poly-commitment/ocaml_types
 
 # Default target
+.PHONY: all
 all: release
 
+.PHONY: setup
 setup: setup-git setup-wasm-toolchain
 
+.PHONY: setup-git
 setup-git:
 		@echo ""
 		@echo "Syncing the Git submodules."
@@ -75,6 +78,7 @@ setup-git:
 		@echo ""
 		@echo "Git submodules synced."
 
+.PHONY: setup-wasm-toolchain
 setup-wasm-toolchain:
 		@ARCH=$$(uname -m); \
 		OS=$$(uname -s | tr A-Z a-z); \
@@ -96,6 +100,7 @@ setup-wasm-toolchain:
 # https://nexte.st/book/pre-built-binaries.html#using-nextest-in-github-actions
 # FIXME: update to 0.9.68 when we get rid of 1.71 and 1.72.
 # FIXME: latest 0.8.19+ requires rustc 1.74+
+.PHONY: install-test-deps
 install-test-deps: ## Install test dependencies
 		@echo ""
 		@echo "Installing the test dependencies."
@@ -107,90 +112,100 @@ install-test-deps: ## Install test dependencies
 		@echo "Test dependencies installed."
 		@echo ""
 
-
+.PHONY: clean
 clean: ## Clean the project
 		@cargo clean
 		@rm -rf $(O1VM_RISCV32IM_BIN_FILES)
 		@rm -rf $(O1VM_MIPS_BIN_DIR)
 
-
+.PHONY: build
 build: ## Build the project
 		cargo build --all-targets --features "$(WORKSPACE_FEATURES)" --workspace \
 			--exclude plonk_wasm --exclude plonk_wasm --exclude xtask
 
-
+.PHONY: release
 release: ## Build the project in release mode
 		cargo build --release --all-targets --features "$(WORKSPACE_FEATURES)" \
 			--workspace --exclude plonk_neon --exclude plonk_wasm --exclude xtask
 
-
+.PHONY: test-doc
 test-doc: ## Test the project's docs comments
 		cargo test --features "$(WORKSPACE_FEATURES)" --release --doc
 
+.PHONY: test-doc-with-coverage
 test-doc-with-coverage:
 		$(COVERAGE_ENV) $(MAKE) test-doc
 
-
+.PHONY: test
 test: ## Test the project with non-heavy tests and using native cargo test runner
 		cargo test --features "$(WORKSPACE_FEATURES)" --release $(CARGO_EXTRA_ARGS) \
 			-- --nocapture --skip heavy $(BIN_EXTRA_ARGS)
 
+.PHONY: test-with-coverage
 test-with-coverage:
 		$(COVERAGE_ENV) CARGO_EXTRA_ARGS="$(CARGO_EXTRA_ARGS)" BIN_EXTRA_ARGS="$(BIN_EXTRA_ARGS)" $(MAKE) test
 
-
+.PHONY: test-heavy
 test-heavy: ## Test the project with heavy tests and using native cargo test runner
 		cargo test --features "$(WORKSPACE_FEATURES)" --release $(CARGO_EXTRA_ARGS) \
 			-- --nocapture heavy $(BIN_EXTRA_ARGS)
 
+.PHONY: test-heavy-with-coverage
 test-heavy-with-coverage:
 		$(COVERAGE_ENV) CARGO_EXTRA_ARGS="$(CARGO_EXTRA_ARGS)" BIN_EXTRA_ARGS="$(BIN_EXTRA_ARGS)" $(MAKE) test-heavy
 
-
+.PHONY: test-all
 test-all: ## Test the project with all tests and using native cargo test runner
 		cargo test --features "$(WORKSPACE_FEATURES)" --release $(CARGO_EXTRA_ARGS) \
 			-- --nocapture $(BIN_EXTRA_ARGS)
 
+.PHONY: test-all-with-coverage
 test-all-with-coverage:
 		$(COVERAGE_ENV) CARGO_EXTRA_ARGS="$(CARGO_EXTRA_ARGS)" BIN_EXTRA_ARGS="$(BIN_EXTRA_ARGS)" $(MAKE) test-all
 
-
+.PHONY: nextest
 nextest: ## Test the project with non-heavy tests and using nextest test runner
 		cargo nextest run --all --features "$(WORKSPACE_FEATURES)" --exclude xtask \
 			--release $(CARGO_EXTRA_ARGS) --profile ci -E "not test(heavy)" $(BIN_EXTRA_ARGS)
 
+.PHONY: nextest-with-coverage
 nextest-with-coverage:
 		$(COVERAGE_ENV) CARGO_EXTRA_ARGS="$(CARGO_EXTRA_ARGS)" BIN_EXTRA_ARGS="$(BIN_EXTRA_ARGS)" $(MAKE) nextest
 
-
+.PHONY: nextest-heavy
 nextest-heavy: ## Test the project with heavy tests and using nextest test runner
 		cargo nextest run --features "$(WORKSPACE_FEATURES)" --release $(CARGO_EXTRA_ARGS) \
 			--profile ci -E "test(heavy)" $(BIN_EXTRA_ARGS)
 
+.PHONY: nextest-heavy-with-coverage
 nextest-heavy-with-coverage:
 		$(COVERAGE_ENV) CARGO_EXTRA_ARGS="$(CARGO_EXTRA_ARGS)" BIN_EXTRA_ARGS="$(BIN_EXTRA_ARGS)" $(MAKE) nextest-heavy
 
-
+.PHONY: nextest-all
 nextest-all: ## Test the project with all tests and using nextest test runner
 		cargo nextest run --features "$(WORKSPACE_FEATURES)" --release $(CARGO_EXTRA_ARGS) \
 			--profile ci $(BIN_EXTRA_ARGS)
 
+.PHONY: nextest-all-with-coverage
 nextest-all-with-coverage:
 		$(COVERAGE_ENV) CARGO_EXTRA_ARGS="$(CARGO_EXTRA_ARGS)" BIN_EXTRA_ARGS="$(BIN_EXTRA_ARGS)" $(MAKE) nextest-all
 
-
+.PHONY: check-format
 check-format: ## Check the code formatting
 		cargo +nightly fmt -- --check
 		taplo fmt --check
 
+.PHONY: format
 format: ## Format the code
 		cargo +nightly fmt
 		taplo fmt
 
+.PHONY: lint
 lint: ## Lint the code
 		cargo clippy --all --features "$(WORKSPACE_FEATURES)" --all-targets --tests \
 			$(CARGO_EXTRA_ARGS) -- -W clippy::all -D warnings
 
+.PHONY: generate-test-coverage-report
 generate-test-coverage-report: ## Generate the code coverage report
 		@echo ""
 		@echo "Generating the test coverage report."
@@ -205,6 +220,7 @@ generate-test-coverage-report: ## Generate the code coverage report
 		@echo "The test coverage report is available at: ./target/coverage"
 		@echo ""
 
+.PHONY: generate-doc
 generate-doc: ## Generate the Rust documentation
 		@echo ""
 		@echo "Generating the documentation."
@@ -215,6 +231,7 @@ generate-doc: ## Generate the Rust documentation
 		@echo "The documentation is available at: ./target/doc"
 		@echo ""
 
+.PHONY: generate-doc-index
 generate-doc-index: ## Generate Rust docs with index page (requires nightly)
 		@echo ""
 		@echo "Generating the documentation with index page."
@@ -226,10 +243,11 @@ generate-doc-index: ## Generate Rust docs with index page (requires nightly)
 		@echo "The documentation is available at: ./target/doc"
 		@echo ""
 
+.PHONY: help
 help: ## Ask for help!
 	@grep -E '^[a-zA-Z0-9_-]+:.*?## .*$$' $(MAKEFILE_LIST) | sort | awk 'BEGIN {FS = ":.*?## "}; {printf "\033[36m%-30s\033[0m %s\n", $$1, $$2}'
 
-
+.PHONY: setup-riscv32-toolchain
 setup-riscv32-toolchain: ## Download and compile the RISC-V 32bits toolchain
 		@echo ""
 		@echo "Setting up the RISC-V 32-bit toolchain"
@@ -243,6 +261,7 @@ setup-riscv32-toolchain: ## Download and compile the RISC-V 32bits toolchain
 		@echo "RISC-V 32-bits toolchain is ready in ${RISCV32_TOOLCHAIN_PATH}/build"
 		@echo ""
 
+.PHONY: build-riscv32-programs
 build-riscv32-programs: setup-riscv32-toolchain ${O1VM_RISCV32IM_BIN_FILES} ## Build all RISC-V 32 bits programs written for the o1vm
 
 ${O1VM_RISCV32IM_BIN_DIR}/%.o: ${O1VM_RISCV32IM_SOURCE_DIR}/%.S
@@ -254,6 +273,7 @@ ${O1VM_RISCV32IM_BIN_DIR}/%.o: ${O1VM_RISCV32IM_SOURCE_DIR}/%.S
 		${RISCV32_TOOLCHAIN_PATH}/build/bin/riscv32-unknown-elf-ld -s -o $(basename $@) $@
 		@echo ""
 
+.PHONY: build-mips-programs
 build-mips-programs: ${O1VM_MIPS_SOURCE_FILES} ${O1VM_MIPS_BIN_FILES} ## Build all MIPS programs written for the o1vm
 
 ${O1VM_MIPS_SOURCE_DIR}/%.asm: ${OPTIMISM_MIPS_SOURCE_DIR}/%.asm
@@ -278,6 +298,7 @@ ${O1VM_MIPS_BIN_DIR}/%.o: ${O1VM_MIPS_SOURCE_DIR}/%.asm
 		@${MIPS_AS} -defsym big_endian=1 -march=mips32r2 -o $@ $<
 		@${MIPS_LD} -s -o $(basename $@) $@
 
+.PHONY: fclean
 fclean: clean ## Clean the tooling artefacts in addition to running clean
 		rm -rf ${RISCV32_TOOLCHAIN_PATH}
 
@@ -294,5 +315,3 @@ build-web: ## Compile the Kimchi library into WebAssembly to be used in the brow
 		--target web \
 		--out-dir ${PLONK_WASM_WEB_OUTDIR} \
 		--rust-version $(NIGHTLY_RUST_VERSION)
-
-.PHONY: all setup install-test-deps clean build release test-doc test-doc-with-coverage test test-with-coverage test-heavy test-heavy-with-coverage test-all test-all-with-coverage nextest nextest-with-coverage nextest-heavy nextest-heavy-with-coverage nextest-all nextest-all-with-coverage format lint generate-test-coverage-report generate-doc generate-doc-index setup-riscv32-toolchain help fclean build-riscv32-programs build-mips-programs check-format