Fix grep commands for BusyBox compatibility
Replace GNU grep --include/--exclude-dir options with find + xargs pattern for BusyBox grep compatibility on self-hosted runners. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
parent
136ef29bd2
commit
d36a46bae8
2 changed files with 4 additions and 9 deletions
|
@ -52,10 +52,7 @@ jobs:
|
|||
- name: Check for console.log statements
|
||||
run: |
|
||||
echo "Checking for console.log statements..."
|
||||
FILES=$(grep -r "console\.log" --include="*.ts" --include="*.js" \
|
||||
--exclude-dir=node_modules --exclude-dir=dist --exclude-dir=public/dist \
|
||||
--exclude-dir=tests --exclude-dir=scripts \
|
||||
src/ || true)
|
||||
FILES=$(find src/ -name "*.ts" -o -name "*.js" | xargs grep -l "console\.log" || true)
|
||||
|
||||
if [ -n "$FILES" ]; then
|
||||
echo "⚠️ Found console.log statements (consider using proper logging):"
|
||||
|
@ -67,9 +64,7 @@ jobs:
|
|||
- name: Check for TODO/FIXME comments
|
||||
run: |
|
||||
echo "Checking for TODO/FIXME comments..."
|
||||
TODOS=$(grep -r "TODO\|FIXME\|HACK\|XXX" --include="*.ts" --include="*.js" \
|
||||
--exclude-dir=node_modules --exclude-dir=dist \
|
||||
. || true)
|
||||
TODOS=$(find . -name "*.ts" -o -name "*.js" | grep -v node_modules | grep -v dist | xargs grep -l "TODO\|FIXME\|HACK\|XXX" || true)
|
||||
|
||||
if [ -n "$TODOS" ]; then
|
||||
echo "📝 Found TODO/FIXME comments:"
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue