nuttxpr commented on PR #14449: URL: https://github.com/apache/nuttx/pull/14449#issuecomment-2429084706
[**\[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 the change necessary? What problem does it solve? * What specific part of the syslog framework was changed? * **Impact:** * Is this a new feature, bug fix, or refactoring? * How does this impact users of the syslog framework? * Are there any compatibility concerns? * **Testing:** * Provide details about the build host and target used for testing. * Include relevant testing logs from *before* and *after* the change to demonstrate the problem and the solution. **Recommendations:** 1. **Expand the Summary:** Explain the reasoning behind removing the LF to CRLF conversion. What issues did it cause? 2. **Clarify the Impact:** Describe the specific changes to the syslog framework. Will users need to make any code adjustments? 3. **Provide Detailed Testing Information:** List the operating systems, architectures, boards, and configurations used for testing. Share complete testing logs to validate the change's effectiveness. **Remember:** A good PR clearly explains the problem, the solution, and how it was validated. Provide as much detail as possible to help reviewers 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
