Skip to content

Commit

Permalink
chore: remove project dir prefix in cleanup function in makefile
Browse files Browse the repository at this point in the history
The `$(PROJECT_DIR)` prefix is unnecessary and can lead to issues if the absolute path contains
whitespace.
  • Loading branch information
jkglasbrenner committed Jul 7, 2023
1 parent 58dcc5f commit c40b3cf
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -214,11 +214,11 @@ define cleanup
$(FIND) . \( -name "__pycache__" -and -not -path "./.tox*" \) -type d -exec $(RM) -rf {} +
$(FIND) . \( -name "*.py[co]" -and -not -path "./.tox*" \) -type f -exec $(RM) -rf {} +
$(FIND) . -name ".ipynb_checkpoints" -type d -exec $(RM) -rf {} +
$(RM) -rf $(PROJECT_DIR)/.coverage
$(RM) -rf $(PROJECT_DIR)/coverage
$(RM) -rf $(PROJECT_DIR)/dist
$(RM) -rf $(PROJECT_DIR)/htmlcov
$(RM) -rf $(PROJECT_DIR)/mlruns
$(RM) -rf .coverage
$(RM) -rf coverage
$(RM) -rf dist
$(RM) -rf htmlcov
$(RM) -rf mlruns
$(RM) -rf $(PROJECT_BUILD_DIR)/bdist*
$(RM) -rf $(PROJECT_BUILD_DIR)/docs
$(RM) -rf $(PROJECT_BUILD_DIR)/lib
Expand Down

0 comments on commit c40b3cf

Please sign in to comment.