linguini1 commented on PR #3219: URL: https://github.com/apache/nuttx-apps/pull/3219#issuecomment-3586481993
> Sure, we have to re check to ensure no any break when upstream, as these change a all small and in apps, merged commits into one pr here. Yes, so can you please tell us what you tested, how you tested and include the log results? > Next pr will make it more clear and more relative and more convenient for reviewer。 No, all PRs need to be clear and convenient for this reviewer. Please update this PR so that it is more clear, as raiden suggested. I really suggest all these commits be broken into multiple PRs, regardless of CI resources, because you have many unrelated fixes for multiple apps and they will all require their own summary, impact and test description. -- 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]
