Repository navigation
search: Suppress headers in quiet mode - #24366
Conversation
If search output is piped to another command, the headers are suppressed and each package name is printed on a separate line. As a user, I had no idea that the headers were suppressed until I looked through the code, so I enabled --quiet flag to force the same behavior.
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Pass quiet mode to result printing Library/Homebrew/cmd/search.rb |
run passes the quiet option to print_results. Formulae and casks use formatted output only when standard output is a TTY and quiet mode is off; otherwise, the results print directly. |
Priority:
Merge Risk: 
Minimal · up to 67c2e
-
The requested quiet search output behavior is implemented without an identified merge-blocking risk.
Pre-merge checks: 
4
-
Full details: Description check
-
Explanation
The description states that piped search output suppresses headers and prints each package name on a separate line, and that the author added a --quiet flag to force that behaviour. This matches the changeset, which passes the quiet flag into print_results and suppresses formatted output when quiet mode is on. The description is related to the change, so the check passes.
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
If search output is piped to another command, the headers are suppressed and each package name is printed on a separate line.
As a user, I had no idea that the headers were suppressed until I looked through the code, so I enabled --quiet flag to force the same behavior.
brew benchmarkresults.brewcommands to reproduce the bug?brew lgtm(style, typechecking and tests) locally?