-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: Rdo 6 1 liner sdk examples #185
Conversation
@John-Memphis can you resolve the conflict? |
@bazen-teklehaymanot I will resolve it. I meant to make this a draft PR |
@John-Memphis all checks should pass before review |
options.Username = "test_user"; | ||
options.Password = Environment.GetEnvironmentVariable("memphis_pass"); | ||
options.Host = "<memphis-host>"; | ||
// options.AccountId = "<memphis-accountId>"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is it in a comment? maybe needs another comment - for cloud users
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shay23b because it errors if I don't put it in a comment. I don't want to put it in a string because it's not a string (it's an int), but I also can't leave it uncommented because it will error.
I looked at the old examples and this was how this was treated to get past the linter complaining about the quotes
No description provided.