Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pkg/paho-mqtt: Update doc.txt #21136

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

DeJusten
Copy link

Add some Information about the paho stack in the doc.txt file

Contribution description

  • Add some Information about the MQTT Version und the runnin behavior

Testing procedure

  • Only documentation, no testing necessary

Issues/PRs references

Add some Information about the paho stack
@github-actions github-actions bot added Area: network Area: Networking Area: doc Area: Documentation Area: pkg Area: External package ports labels Jan 14, 2025
@benpicco benpicco changed the title Update doc.txt pkg/paho-mqtt: Update doc.txt Jan 14, 2025
@benpicco benpicco requested a review from OlegHahm January 14, 2025 18:12
pkg/paho-mqtt/doc.txt Show resolved Hide resolved
pkg/paho-mqtt/doc.txt Outdated Show resolved Hide resolved
pkg/paho-mqtt/doc.txt Outdated Show resolved Hide resolved
@OlegHahm
Copy link
Member

Thanks for the contribution and improvement - and welcome to RIOT!

And sorry for my somewhat pedantic review: actually I thought to review a PR my a fellow RIOT maintainer. 😅

@MrKevinWeiss
Copy link
Contributor

ping @DeJusten I think some simple fixes are needed to get in.

DeJusten and others added 2 commits January 30, 2025 09:49
Thanks for correcting. Next time I will pay attention to correct pronunciation

Co-authored-by: Oleg Hahm <[email protected]>
Also thanks for the correction

Co-authored-by: Oleg Hahm <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: doc Area: Documentation Area: network Area: Networking Area: pkg Area: External package ports
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants