Skip to content

feat(builtins): add grep -L, --exclude-dir, -s, -Z flags#385

Merged
chaliy merged 1 commit intomainfrom
claude/fix-380-383-grep-flags-Vvs93
Feb 27, 2026
Merged

feat(builtins): add grep -L, --exclude-dir, -s, -Z flags#385
chaliy merged 1 commit intomainfrom
claude/fix-380-383-grep-flags-Vvs93

Conversation

@chaliy
Copy link
Copy Markdown
Contributor

@chaliy chaliy commented Feb 27, 2026

Summary

Test plan

  • Unit tests for each flag (positive and negative cases)
  • Spec tests for each flag in grep.test.sh
  • cargo fmt --check passes
  • cargo clippy --all-targets --all-features -- -D warnings passes
  • cargo test --all-features -p bashkit --lib passes (1126 tests)
  • cargo test --all-features -p bashkit --test spec_tests -- grep passes (82 spec tests)

@chaliy chaliy force-pushed the claude/fix-380-383-grep-flags-Vvs93 branch from 2eed0e5 to 8aebcd9 Compare February 27, 2026 23:14
- -L/--files-without-match: print filenames with zero matches (#380)
- --exclude-dir=GLOB: skip directories during recursive grep (#381)
- -s/--no-messages: suppress error messages for nonexistent files (#382)
- -Z/--null: use \0 after filenames instead of : or \n (#383)

Closes #380, closes #381, closes #382, closes #383
@chaliy chaliy force-pushed the claude/fix-380-383-grep-flags-Vvs93 branch from 8aebcd9 to 4b3eabe Compare February 27, 2026 23:41
@chaliy chaliy merged commit 0f620ba into main Feb 27, 2026
16 checks passed
@chaliy chaliy deleted the claude/fix-380-383-grep-flags-Vvs93 branch March 12, 2026 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants