-
Notifications
You must be signed in to change notification settings - Fork 187
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
Moved KMS code examples into KMS directory; added new KMS code examples #480
Conversation
|
||
let conf = Config::builder().region(region).build(); | ||
let conn = aws_hyper::conn::Standard::https(); | ||
let client = Client::from_conf_conn(conf, conn); |
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.
I'm thinking we should have most of examples instantiate clients without customizing the connector, like this:
let config = Config::builder().region(region).build();
let client = Client::from_conf(config);
That should show people how to build a client quickly and easily without getting overloaded with details. At the same time, we should have an example somewhere with some good documentation on how to customize the HTTP connection, and maybe some concrete examples of why you would want to do that.
What do you think?
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.
I agree. I'll make the necessary changes and clear the "waiting-on-author" bit when I'm done.
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.
Done.
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.
LGTM!
Issue #, if available:
KMS had only one code example, in the kms-helloworld directory.
Description of changes:
Moved the helloword code into the kms/src/bin directory; added another set of KMS code examples.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.