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

Rdo 6 1 liner sdk examples #243

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

Conversation

John-Memphis
Copy link
Contributor

No description provided.

@idanasulin2706
Copy link
Contributor

Please change the examples in the examples directory as well, in all sdks

@idanasulin2706
Copy link
Contributor

Quick start should be the shortest way to send/receive messages.
For producing you can simply create a connection and use it to produce messages without creating a producer first, same for consuming, so make sure you are fixing it where needed in other PRs as well

shay23b
shay23b previously requested changes Jan 4, 2024
examples/consumer.py Outdated Show resolved Hide resolved
examples/consumer.py Outdated Show resolved Hide resolved
examples/consumer.py Outdated Show resolved Hide resolved
examples/consumer.py Outdated Show resolved Hide resolved
examples/consumer.py Outdated Show resolved Hide resolved
examples/consumer.py Outdated Show resolved Hide resolved
examples/producer.py Outdated Show resolved Hide resolved
@John-Memphis John-Memphis dismissed shay23b’s stale review January 13, 2024 19:27

Fixes were implemented

@John-Memphis
Copy link
Contributor Author

3.8 fails because of using new type hinting syntax that became available in 3.9. Should I go back to syntax that is available in 3.8?

host = "<memphis-host>",
username = "<memphis-username>",
password = "<memphis-password>",
account_id = <mmephis-account-id> # For cloud users, at the top of the overview page
Copy link
Contributor

Choose a reason for hiding this comment

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

memphis

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@shay23b Same issue with the comment in the C# branch, where this fails linting when I don't comment out that.

Is that okay? I don't know how to change the rules so it allows that.

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.

3 participants