Skip to content

Commit 409f20f

Browse files
authored
fix: include virtual environment in clean target (#335)
* fix: include virtual environment in clean target - Add to DIRS_TO_CLEAN variable - This ensures 'make clean' removes the dev environment created via 'make dev' - Fixes issue where clean target didn't clean up virtual environment * fixed make clean to clear dev environment and remove database files
1 parent acd1619 commit 409f20f

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

Makefile

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ TEST_DOCS_DIR ?= $(DOCS_DIR)/docs/test
2020
# Project-wide clean-up targets
2121
DIRS_TO_CLEAN := __pycache__ .pytest_cache .tox .ruff_cache .pyre .mypy_cache .pytype \
2222
dist build site .eggs *.egg-info .cache htmlcov certs \
23-
$(VENV_DIR).sbom $(COVERAGE_DIR) \
23+
$(VENV_DIR) $(VENV_DIR).sbom $(COVERAGE_DIR) \
2424
node_modules
2525

2626
FILES_TO_CLEAN := .coverage coverage.xml mcp.prof mcp.pstats \
@@ -29,7 +29,8 @@ FILES_TO_CLEAN := .coverage coverage.xml mcp.prof mcp.pstats \
2929
$(DOCS_DIR)/pstats.png \
3030
$(DOCS_DIR)/docs/test/sbom.md \
3131
$(DOCS_DIR)/docs/test/{unittest,full,index,test}.md \
32-
$(DOCS_DIR)/docs/images/coverage.svg $(LICENSES_MD) $(METRICS_MD)
32+
$(DOCS_DIR)/docs/images/coverage.svg $(LICENSES_MD) $(METRICS_MD) \
33+
*.db *.sqlite *.sqlite3 mcp.db-journal
3334

3435
COVERAGE_DIR ?= $(DOCS_DIR)/docs/coverage
3536
LICENSES_MD ?= $(DOCS_DIR)/docs/test/licenses.md
@@ -156,7 +157,7 @@ certs: ## Generate ./certs/cert.pem & ./certs/key.pem
156157
chmod 640 certs/key.pem
157158

158159
## --- House-keeping -----------------------------------------------------------
159-
# help: clean - Remove caches, build artefacts, virtualenv, docs, certs, coverage, SBOM, etc.
160+
# help: clean - Remove caches, build artefacts, virtualenv, docs, certs, coverage, SBOM, database files, etc.
160161
.PHONY: clean
161162
clean:
162163
@echo "🧹 Cleaning workspace..."

0 commit comments

Comments
 (0)