nuttxpr commented on PR #14494: URL: https://github.com/apache/nuttx/pull/14494#issuecomment-2434594926
[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) This PR appears **incomplete** based on the provided information. **Here's why:** * **Summary:** Lacks details. * **Missing:** * **Reason for change:** Is this a fix, update, or new feature? Why are READMEs being migrated? * **Specifics:** Which boards are affected by the README migration? What kind of "fixes" are being made to MPS boards? * **Impact:** Too vague. * **"readmes in documentation" is not informative enough.** Specify *how* the documentation is impacted. Are there new instructions? Removed content? * **Testing:** Insufficient. * **Local build is not enough.** Which boards were built? What were the results? * **No logs provided.** "Testing logs before/after" sections are empty. **To improve this PR:** 1. **Expand the Summary:** Clearly explain the problem this PR addresses and the solution it implements. 2. **Detail the Impact:** Be specific about changes to documentation. What should users know? 3. **Provide Thorough Testing Information:** * List the build host OS, CPU, and compiler used. * Specify the target architectures and boards tested. * Include relevant snippets from build logs (both before and after the change) to demonstrate the issue and the fix. A well-documented PR makes it easier for reviewers to understand and approve your changes. -- 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
