nuttxpr commented on PR #2714:
URL: https://github.com/apache/nuttx-apps/pull/2714#issuecomment-2408833132

   [**\[Experimental Bot, please feedback 
here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues)
   
   
   
   **No, this PR does not meet the NuttX requirements.**
   
   **Missing Information:**
   
   * **Summary:**
       *  Lacks a description of why the change is necessary. Is it a new 
feature request? Does it fix a bug? 
       *  Needs more detail on how the "top" and "watch" commands work. What 
information do they display? How are they used?
   * **Impact:** 
       *  All impact sections are empty.  You must address each point (user 
impact, build impact, hardware impact, etc.) with either "NO" or a "YES" 
followed by a description.
   * **Testing:**
       *  "sim" is not enough information. Specify the  host OS, CPU, compiler, 
target architecture, board, and configuration.
       *  Provide actual testing logs from before and after the change.
   
   **To improve this PR:**
   
   1. **Expand the Summary:** Clearly articulate the purpose and functionality 
of the new commands.
   2. **Complete the Impact Section:** Assess and describe the impact of your 
changes on all aspects mentioned.
   3. **Provide Detailed Testing Information:** List your testing environment 
specifics and include relevant logs demonstrating the changes' effects. 
   


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