ethanlin01x opened a new pull request, #3792: URL: https://github.com/apache/iggy/pull/3792
## Which issue does this PR address? Closes #3791 ## Rationale The Rust and C# preflights reported success without executing a single CLI command from the root README; see the issue for the full analysis. ## What changed? The preflight grep still targeted `cargo r` while the README documents `cargo run`, and GNU grep parses `` \` `` in an ERE as a buffer-start anchor rather than a literal backtick, so the extraction was always empty and the runner silently skipped it. The pattern now matches the backtick via a bracket expression, and the transform substitutes the README's credential placeholders with the test server's credentials so the commands survive `eval`. The C# preflight is removed instead of fixed: it duplicated the Rust job, which is the only one that builds the Rust CLI. A second commit normalizes the remaining `cargo r` occurrences in the example docs. ## Local Execution - Passed - Pre-commit hooks ran ## AI Usage If AI tools were used, please answer: 1. Which tools? Claude code 2. Scope of usage? help to analysis and generate PR comment 3. How did you verify the generated code works correctly? Ran the fixed preflight end to end against a real local server 4. Can you explain every line of the code if asked? Yes, all the changes are checked by the human. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
