Skip to content
This repository was archived by the owner on Oct 29, 2023. It is now read-only.

Conversation

@gcseh
Copy link
Contributor

@gcseh gcseh commented Sep 22, 2021

Fix for #188

@product-auto-label product-auto-label bot added the api: cloudiot Issues related to the googleapis/python-iot API. label Sep 22, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 22, 2021
@gcseh gcseh marked this pull request as ready for review September 22, 2021 14:04
@gcseh gcseh requested review from a team as code owners September 22, 2021 14:04
@gcseh gcseh force-pushed the mqtt-example-nox branch 2 times, most recently from bd6a208 to e715c14 Compare September 22, 2021 14:34
@gcseh gcseh assigned gcseh and unassigned joeklemm Sep 22, 2021
@tseaver tseaver changed the title docs: Add README for MQTT example docs: sample for MQTT Sep 22, 2021
@tseaver tseaver changed the title docs: sample for MQTT docs: add README, nox support for MQTT example Sep 22, 2021
Copy link
Contributor

@dandhlee dandhlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see my comments below!

As well, docs prefix triggers a patch version bump, do we want that happening from this PR?

@gcseh gcseh changed the title docs: add README, nox support for MQTT example chore: add README, nox support for MQTT example Sep 24, 2021
@dandhlee
Copy link
Contributor

Discussed offline, closing the loop here; see example on adding temporary OwlBot changes to make it happy but also continue receiving template updates when needed.

@gcseh gcseh added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 27, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 27, 2021
Copy link
Contributor

@dandhlee dandhlee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, please see two actions items below.

@gcseh gcseh merged commit 040cee1 into main Sep 27, 2021
@gcseh gcseh deleted the mqtt-example-nox branch September 27, 2021 16:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: cloudiot Issues related to the googleapis/python-iot API. cla: yes This human has signed the Contributor License Agreement.

4 participants