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
Updated pre-commit.sh to check for unused strings in strings.en.yaml #4596
base: master
Are you sure you want to change the base?
Changes from 1 commit
26c9145
40cf4c6
8200bce
171781c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,9 @@ | ||
#!/bin/sh | ||
"$(dirname "$0")/_/husky.sh" | ||
|
||
MATCH_PATH='chaoscenter/web/' | ||
echo "\033[0;31;1mYou can't commit directly to main branch\033[0m" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this tested properly? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have not changed the code for this, it is how the original implementation is in pre-commit.sh. Should I change it to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. hey @shruti2522 that went unnoticed by me. Please change it to master. |
||
|
||
MATCH_PATH='chaoscenter/web/' | ||
BRANCH=$(git rev-parse --abbrev-ref HEAD) | ||
|
||
if [ "$BRANCH" = "main" ]; then | ||
|
@@ -16,12 +17,20 @@ if expr "$BRANCH" : 'release\/.*' >/dev/null; then | |
fi | ||
|
||
LITMUS_UI_FILE_CHANGES=$(git diff --name-only --cached | grep "$MATCH_PATH" -c) | ||
|
||
echo "\033[0;93mLITMUS_UI_FILE_CHANGES\033[0m ⟶ $LITMUS_UI_FILE_CHANGES" | ||
|
||
if [ "$LITMUS_UI_FILE_CHANGES" -gt 0 ]; | ||
then | ||
cd ./chaoscenter/web && yarn lint-staged | ||
else | ||
echo "\033[0;93mSkipping husky pre-commit hook in $MATCH_PATH folder\033[0m" | ||
fi | ||
if [ "$LITMUS_UI_FILE_CHANGES" -gt 0 ]; then | ||
cd ./chaoscenter/web && yarn lint-staged | ||
else | ||
echo "\033[0;93mSkipping husky pre-commit hook in $MATCH_PATH folder\033[0m" | ||
fi | ||
|
||
# check for unused strings in strings.en.yaml | ||
UNUSED_STRINGS=$(grep -Fvxf <(grep -oP '"\K[^"]+' ./chaoscenter/web/strings/strings.en.yaml) <(grep -rl 'chaoscenter/web/src' ./*)) | ||
if [ "$UNUSED_STRINGS" != "" ]; then | ||
echo "\033[0;31;1mERROR: Unused strings found in strings.en.yaml\033[0m" | ||
echo "$UNUSED_STRINGS" | ||
exit 1 | ||
fi | ||
|
||
exit 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this line correct? this will be printed for each pre-commit hook irrespective to the branch