nuttxpr commented on PR #15112: URL: https://github.com/apache/nuttx/pull/15112#issuecomment-2531463952
[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) No. This PR summary is insufficient. While it states the reason *why* the change is needed, it does not describe *what* is being changed, *how* it is being changed, or which functional parts of the code are affected. Also, references to related issues/PRs are missing. The Impact section is inadequate. "RELEASE" suggests a significant impact, which necessitates detailed explanations for each impact category. Simply stating "RELEASE" does not offer any specifics. The Testing section is also inadequate. "CI" implies reliance solely on continuous integration testing. However, local testing verification is required, with specific details about the host and target environments as well as *before* and *after* testing logs. Just stating "CI" is not acceptable. -- 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
