Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Pre Commit Check #4642

Merged
merged 8 commits into from
Nov 22, 2024
Merged
26 changes: 23 additions & 3 deletions chaoscenter/web/.husky/pre-commit
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,31 @@
"$(dirname "$0")/_/husky.sh"

MATCH_PATH='chaoscenter/web/'
STRINGS_FILE="chaoscenter/web/strings/strings.en.yaml"

check_and_delete_unused_strings() {
TEMP_FILE=$(mktemp)
grep -rEo '\b[a-zA-Z0-9_]+\b' chaoscenter/web/**/*.js > "$TEMP_FILE"
UNUSED_STRINGS=$(grep -v '^#' "$STRINGS_FILE" | grep -vFf "$TEMP_FILE" | grep -oE '^[a-zA-Z0-9_]+')
rm "$TEMP_FILE"

if [ -n "$UNUSED_STRINGS" ]; then
echo "Unused strings found: $UNUSED_STRINGS"
for UNUSED_STRING in $UNUSED_STRINGS; do
sed -i "/$UNUSED_STRING:/d" "$STRINGS_FILE"
done
echo "Unused strings deleted from $STRINGS_FILE"
else
echo "No unused strings found in $STRINGS_FILE"
fi
}

check_and_delete_unused_strings

BRANCH=$(git rev-parse --abbrev-ref HEAD)

if [ "$BRANCH" = "main" ]; then
echo "\033[0;31;1mYou can't commit directly to main branch\033[0m"
if [ "$BRANCH" = "master" ]; then
echo "\033[0;31;1mYou can't commit directly to master branch\033[0m"
exit 1
fi

Expand All @@ -24,4 +44,4 @@ if [ "$LITMUS_UI_FILE_CHANGES" -gt 0 ];
cd ./chaoscenter/web && yarn lint-staged
else
echo "\033[0;93mSkipping husky pre-commit hook in $MATCH_PATH folder\033[0m"
fi
fi
Loading