nuttxpr commented on PR #13619: URL: https://github.com/apache/nuttx/pull/13619#issuecomment-2375702136
[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This PR summary is missing crucial information: **Missing in Summary:** * **Why is the change necessary?** Simply stating "fix" isn't helpful. Explain the bug, the desired improvement, or the need for the new feature. * **How does the change work?** Provide a brief technical explanation of your solution. **Missing in Impact:** * **All sections are empty.** You **must** address each impact area. Even if the answer is "NO", state it explicitly. For example: * "Impact on user: NO" * "Impact on build: NO" **Missing in Testing:** * **Provide actual testing logs.** "your testing logs here" is not helpful. Include relevant snippets of logs demonstrating the issue before and the successful outcome after your changes. * **Specify your testing environment.** Detail the build host OS, CPU, compiler, target architecture, board, and configuration used for testing. **In addition to filling in the missing information, consider these suggestions:** * **Conciseness:** While the headings are good, the descriptions can be more concise. * **Clarity:** Use descriptive language to explain the changes and their effects. By providing this missing information, you'll make it much easier for reviewers to understand and assess your PR, significantly increasing its chances of being accepted. -- 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]
