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

Rewrite simple python client explanation. #3998

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

clalancette
Copy link
Contributor

This makes it look much more like the service section, with more explanation. It also gets rid of an inconsistency where it was referencing a while loop that is no longer there.

This should fix #3997

This makes it look much more like the service section,
with more explanation.  It also gets rid of an inconsistency
where it was referencing a while loop that is no longer
there.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
@clalancette clalancette added the backport-all backport at reviewers discretion; from rolling to all versions label Nov 2, 2023
@clalancette clalancette merged commit 53921ee into rolling Nov 2, 2023
3 checks passed
@clalancette clalancette deleted the clalancette/service-and-client-fix branch November 2, 2023 13:13
mergify bot pushed a commit that referenced this pull request Nov 2, 2023
This makes it look much more like the service section,
with more explanation.  It also gets rid of an inconsistency
where it was referencing a while loop that is no longer
there.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit 53921ee)
mergify bot pushed a commit that referenced this pull request Nov 2, 2023
This makes it look much more like the service section,
with more explanation.  It also gets rid of an inconsistency
where it was referencing a while loop that is no longer
there.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit 53921ee)
clalancette added a commit that referenced this pull request Nov 2, 2023
This makes it look much more like the service section,
with more explanation.  It also gets rid of an inconsistency
where it was referencing a while loop that is no longer
there.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit 53921ee)

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
clalancette added a commit that referenced this pull request Nov 2, 2023
This makes it look much more like the service section,
with more explanation.  It also gets rid of an inconsistency
where it was referencing a while loop that is no longer
there.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
(cherry picked from commit 53921ee)

Co-authored-by: Chris Lalancette <clalancette@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-all backport at reviewers discretion; from rolling to all versions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Client code discussion is out of date in tutorial
2 participants