zs39 commented on PR #3255: URL: https://github.com/apache/nuttx-apps/pull/3255#issuecomment-3668841087
> Thank you @zs39 very nice addition mqtt is very popular! :-) > > * Please check CI errors, cmake formatting needs fix. > * Could you please provide testing logs before and after the change? If this is "modification" then something was not working before right? If this is "introducing mqtt tests" then its a new feature not present before? > * Thank you for providing logs where tests fail. Could you please provide logs where tests pass? This patch only compiles the native MQTT test cases; they weren't originally compiled and run. The native test cases themselves might have inherent issues, which won't be fixed in this patch.@cederom -- 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]
