Skip to content

mqtt support#16

Open
kefalle wants to merge 1 commit into
m5stack:mainfrom
kefalle:mqtt-support
Open

mqtt support#16
kefalle wants to merge 1 commit into
m5stack:mainfrom
kefalle:mqtt-support

Conversation

@kefalle

@kefalle kefalle commented May 10, 2026

Copy link
Copy Markdown

No description provided.

@Ocean-lhy Ocean-lhy requested a review from hlym123 June 8, 2026 06:30
@Ocean-lhy

Copy link
Copy Markdown
Contributor

Hi @kefalle,

Thank you for your contribution and welcome to the project! We really appreciate you taking the time to add MQTT support.

However, I noticed that the PR description is currently empty. Because this is a relatively large feature (modifying 10 files), it's difficult for us to review the code effectively without some context.

Could you please edit the PR description and provide some details? It would be very helpful to include:

Motivation: What specific use cases does this MQTT support address?
Implementation details: A brief overview of how it works.
Testing: How did you test this feature?
Additionally, since the main branch has recently been updated to v0.2.0, this PR currently has merge conflicts. You will need to pull the latest changes from main and resolve these conflicts first.

Once the description is updated and the conflicts are resolved, please let us know, and we will be happy to proceed with a detailed code review. Let me know if you need any help with the merge conflicts!

Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants