nuttxpr commented on PR #13544: URL: https://github.com/apache/nuttx/pull/13544#issuecomment-2360711736
[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13494&type=pullrequests) **No, this PR does not fully meet the NuttX requirements.** **Here's why and how to improve it:** * **Insufficient Summary:** * **Why?** The summary lacks a clear explanation of *why* adding this configuration for the extended sentence is necessary. What problem does it solve? What benefit does it provide? * **What functional part?** Be more specific than "gnssutils." Which file(s) are modified? * **Missing Impact Details:** * **New feature?** Is this a new feature, or is it enabling an existing feature that was previously unavailable? * **User impact?** Will users need to change any configurations or will this "just work"? * **Build/Hardware/Documentation/Security/Compatibility:** You've marked "gnssutils" under impact, but you need to address each specific impact category with "YES/NO" and provide details if applicable. * **Incomplete Testing:** * **Build Host Details:** Provide specifics about your build host environment (OS, CPU architecture, compiler version). * **Target Details:** Specify the target architecture (e.g., "sim" for simulator, "ARMv7-M") and the board configuration you used for testing. * **Insufficient Logs:** The provided testing logs are too vague. Include relevant snippets of logs *before* and *after* your changes, clearly demonstrating the issue and the fix. For example: * **Before:** Show that the extended sentence was NOT being parsed. * **After:** Show that the extended sentence IS now being parsed correctly. **Example of Improved Summary:** ``` ## Summary This PR enables support for the vendor-specific "LOR LSQ" extended sentence in the MINMEA NMEA library. This sentence provides [explain the data provided by the sentence and why it's useful]. The change adds a new configuration option to enable parsing of this sentence and updates the MINMEA library to handle the new sentence type. Related NuttX Issue: [If applicable, link to an issue discussing the need for this feature] ``` **Remember:** A good PR description saves reviewers time and helps ensure your changes are accepted 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]
