From 7d8bf4eaf57a43d07e8e2eb5d9f23839cf3c4a64 Mon Sep 17 00:00:00 2001 From: Arnav Choudhury Date: Sat, 6 Apr 2024 13:53:28 +0530 Subject: [PATCH] Added a pre-commit check to fail in case user forgot to commit migration files --- .../.pre-commit-config.yaml | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/{{cookiecutter.project_slug}}/.pre-commit-config.yaml b/{{cookiecutter.project_slug}}/.pre-commit-config.yaml index ece70ffe7e..f1f195553f 100644 --- a/{{cookiecutter.project_slug}}/.pre-commit-config.yaml +++ b/{{cookiecutter.project_slug}}/.pre-commit-config.yaml @@ -49,6 +49,19 @@ repos: - id: djlint-reformat-django - id: djlint-django + - repo: local + hooks: + - id: check-migrations + name: check-migrations + {%- if cookiecutter.use_docker == 'y' %} + entry: /usr/bin/env bash -c "bash .envs/.local/.django && bash .envs/.local/.postgres && docker compose -f local.yml run --rm django bash -c 'python manage.py makemigrations --check --dry-run'" + {%- else %} + entry: /usr/bin/env bash -c "bash .envs/.local/.django && bash .envs/.local/.postgres && python manage.py makemigrations --check --dry-run" + {%- endif %} + types: [ python ] + pass_filenames: false + language: system + # sets up .pre-commit-ci.yaml to ensure pre-commit dependencies stay up to date ci: autoupdate_schedule: weekly