linguini1 commented on PR #17075: URL: https://github.com/apache/nuttx/pull/17075#issuecomment-3352826319
I find it disrespectful that this PR was merged despite three separate reviewers requesting additional logs from hardware tests. @acassis @jerpelea and myself requested hardware test logs, and @wangchdo stated they think it would be worth running further tests. My review where I requested those changes specifically was dismissed. If you tested on hardware regularly, as you say, include the logs. It is a contribution requirement. Something needs to be done to prevent the ability here of just dismissing those requests for logs and merging anyways. -- 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]
