nuttxpr commented on PR #14064: URL: https://github.com/apache/nuttx/pull/14064#issuecomment-2405210979
[**\[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:** * **Why** is this change necessary? What problem does it solve? * **What** specific module compilation options are being unified? * **How** are the options being unified? What is the new, consistent approach? * **Impact:** * Provide specific details for each impact category (e.g., which architectures, boards, drivers are affected). * Instead of just stating "risc-v," explain the impact on RISC-V. * **Testing:** * **Insufficient detail:** "Build it and run it with qemu" is not descriptive enough. * **No logs:** Provide actual testing logs from before and after the change. **Recommendations:** 1. **Expand the Summary:** Clearly articulate the problem, the solution, and the technical details of the change. 2. **Complete the Impact Section:** Address each category with specific information. If an impact is "NO," briefly explain why. 3. **Provide Detailed Testing Information:** * Specify the exact build host and target environments. * Include relevant commands used for building and testing. * Share actual log output that demonstrates the change's effect. A well-structured PR with complete information will significantly increase its chances of being reviewed and merged quickly. -- 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]
