From 57834ef5519d39ce51c46761d5448bb9a8ddeb08 Mon Sep 17 00:00:00 2001 From: Miroslav Shubernetskiy Date: Thu, 5 Oct 2023 22:02:37 -0400 Subject: [PATCH] adding pre-commit hook for checking chalk version across files this will check chalk version matches across various files. currently we only check: * nimble file * base_keyspecs.c4m If the chalk version is updated in nimble file, all other files need to match it. --- .github/check_version.sh | 22 ++++++++++++++++++++++ .pre-commit-config.yaml | 10 ++++++++++ 2 files changed, 32 insertions(+) create mode 100755 .github/check_version.sh diff --git a/.github/check_version.sh b/.github/check_version.sh new file mode 100755 index 00000000..e6be909f --- /dev/null +++ b/.github/check_version.sh @@ -0,0 +1,22 @@ +#!/usr/bin/env bash +set -eEu +set -o pipefail + +FILEDIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)" +cd "$FILEDIR" 1> /dev/null 2>&1 + +# cd to root of the repo +cd .. + +KEYSPEC=src/configs/base_keyspecs.c4m +current_version=$(make version) +keyspec_version=$( + grep -E "chalk_version\s:=" $KEYSPEC \ + | cut -d'"' -f2 +) + +if [ "$current_version" != "$keyspec_version" ]; then + echo $KEYSPEC chalk_version does not match nimble chalk version + echo "$keyspec_version != $current_version" + exit 1 +fi diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 67a2feab..f498244f 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -32,6 +32,16 @@ repos: data/.* )$ + - repo: local + hooks: + - id: chalkversion + name: chalkversion + description: verify chalk version matches across all chalk files + entry: ./.github/check_version.sh + language: system + always_run: true + pass_filenames: false + - repo: https://github.com/tcort/markdown-link-check rev: v3.11.2 hooks: