xiaoxiang781216 commented on PR #17849: URL: https://github.com/apache/nuttx/pull/17849#issuecomment-3776533041
> Ok, what do you propose as the next steps? I’m finishing all tests locally and everything seems to be working. There will still be some fixes needed in NuttX and NuttX apps — for example, functions returning a value even though they’re declared void, and an issue with include guard naming in NuttX apps. > > How do you want to handle merging? Should I separate all build-related fixes into a dedicated PR's, and then submit another PR later with the updated Dockerfile? > Yes, it's better to merge the nuttx/apps change first. -- 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]
