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

   [**\[Experimental Bot, please feedback 
here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues)
   
   
   
   This PR template itself meets the NuttX requirements by providing clear 
sections for all the requested information.  However, a *completed* PR needs 
the placeholder text within each section replaced with the specific details of 
the change being proposed.  Simply copying and pasting this template without 
filling in the details is **not** sufficient.
   
   
   ## Summary
   
   * The current template clearly requests the reason for the change (fix, 
update, new feature), the affected functional area, the mechanics of the 
change, and links to related issues.  This addresses the summary requirements.
   
   ## Impact
   
   * The template explicitly asks for the impact on the user, build process, 
hardware, documentation, security, compatibility, and other considerations. 
This fully covers the impact requirements.
   
   ## Testing
   
   * The template requests confirmation of testing, specifies the desired 
information about the build host and target, and provides placeholders for 
before-and-after testing logs.  This addresses the testing requirements, 
although contributors must remember to actually *include* the logs and 
host/target details.
   
   
   In short, the template itself is well-structured to meet the requirements.  
It is the responsibility of the PR submitter to populate it with meaningful 
information.
   


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