Skip to content

Commit 6fb2cd6

Browse files
rtibblesclaude
andcommitted
chore(makefile): fix tab indent and add make -n pre-commit guard
The deploy-migrate recipe was indented with 8 spaces instead of a tab, which would fail at parse time the next time the target ran. The new make -n pre-commit hook catches this class of parse error locally before it can land. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent f443ef1 commit 6fb2cd6

2 files changed

Lines changed: 10 additions & 1 deletion

File tree

.pre-commit-config.yaml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,15 @@ repos:
8181
contentcuration/kolibri_public/migrations/0004_auto_20240612_1847.py|
8282
contentcuration/kolibri_public/migrations/0006_auto_20250417_1516.py|
8383
)$
84+
# Only checks the root Makefile. Extend if nested Makefiles get added.
85+
- repo: local
86+
hooks:
87+
- id: makefile-syntax
88+
name: Makefile syntax check
89+
entry: make -n
90+
language: system
91+
files: ^Makefile$
92+
pass_filenames: false
8493
# Always keep black as the final hook so it reformats any other reformatting.
8594
- repo: https://github.com/python/black
8695
rev: 20.8b1

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ migrate:
3939
# 4) Remove the management command from this `deploy-migrate` recipe
4040
# 5) Repeat!
4141
deploy-migrate:
42-
echo "Nothing to do here!"
42+
echo "Nothing to do here!"
4343

4444
contentnodegc:
4545
python contentcuration/manage.py garbage_collect

0 commit comments

Comments
 (0)