chennaji9 commented on PR #5209:
URL: 
https://github.com/apache/rocketmq-dashboard/pull/5209#issuecomment-6227378633

   Re-verified against the current tip: the text branch in 
`rmqctl/cmd/version.go` still ignores every `fmt.Fprintf` result and returns 
`nil`, so the asymmetry with the JSON/YAML paths is still live on today's base. 
One additional angle worth noting for the record: the ignored error is 
reachable through a broken pipe too — if something like `rmqctl version | head 
-1` closes stdout early, the remaining line writes fail with EPIPE and the 
command still exits 0, so a script consuming partial output can't tell the 
print was truncated. The branch applies cleanly after the recent merges.


-- 
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]

Reply via email to