nuttxpr commented on PR #15337:
URL: https://github.com/apache/nuttx/pull/15337#issuecomment-2561673054

   [**\[Experimental Bot, please feedback 
here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues)
   
   
   
   No.  This PR summary is insufficient. It states *what* was changed but not 
*why*.  It does not explain the problem that the change solves, the functional 
part of the code being modified, or *how* the change works. The Impact section 
only references a CID, which is not publicly accessible information.  It needs 
to describe the impacts in the requested format. The testing section needs 
actual testing logs, not just "CI".  The PR author needs to provide 
before/after logs demonstrating the fix.
   


-- 
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: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to