The GitHub Actions job "SonarCloud Analysis" on kvrocks.git/unstable has 
succeeded.
Run started by GitHub user git-hulk (triggered by git-hulk).

Head commit for run:
3b02893fbcd1e30b707f6d2f09fda01e63394551 / Sanjana <[email protected]>
refactor: use stderr for CLI error output (#3372)

I noticed that the CLI was printing error messages (like invalid
arguments or config load failures) to `stdout`, which isn't ideal for
scripting or standard error handling. This PR redirects those critical
errors to `stderr`.

I also enhanced the
[PrintUsage](cci:1://file:///Users/sanjana./kvrocks/src/cli/main.cc:62:0-73:1)
function to accept an output stream, so `-h`/`--help` still goes to
`stdout` (as expected), but incorrect usage warnings go to `stderr`. The
usage message itself has been slightly polished for better readability.

Changes:
- PrintUsage now takes an optional `std::ostream&` argument (defaults to
`cout`).
- Invalid command-line arguments now print usage to `cerr` and return
exit code 1.
- Config loading and logger initialization errors now print to `cerr`.
- Added `<iostream>` include for correctness.

---------

Co-authored-by: 纪华裕 <[email protected]>

Report URL: https://github.com/apache/kvrocks/actions/runs/23011275057

With regards,
GitHub Actions via GitBox

Reply via email to