From f003000d8871cb0a5b2f87c3e705d2bb7f85a130 Mon Sep 17 00:00:00 2001 From: Landon James Date: Fri, 3 Jan 2025 10:06:50 -0700 Subject: [PATCH] Update docs for non aws-config config builders (#3960) ## Motivation and Context Partially addresses https://github.com/awslabs/aws-sdk-rust/issues/1226, although we might want to look into going further and actually deprecating or warning on these builder methods. ## Description ## Testing Doc only update ---- _By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice._ --- .../generators/config/ServiceConfigGenerator.kt | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/codegen-client/src/main/kotlin/software/amazon/smithy/rust/codegen/client/smithy/generators/config/ServiceConfigGenerator.kt b/codegen-client/src/main/kotlin/software/amazon/smithy/rust/codegen/client/smithy/generators/config/ServiceConfigGenerator.kt index bca2f15c6e..a796b65da6 100644 --- a/codegen-client/src/main/kotlin/software/amazon/smithy/rust/codegen/client/smithy/generators/config/ServiceConfigGenerator.kt +++ b/codegen-client/src/main/kotlin/software/amazon/smithy/rust/codegen/client/smithy/generators/config/ServiceConfigGenerator.kt @@ -399,6 +399,13 @@ class ServiceConfigGenerator( } fun render(writer: RustWriter) { + val configDocs = """ + Constructs a config builder. +
+ Note that a config created from this builder will not have the same safe defaults as one created by + the aws-config crate. +
+ """ writer.docs("Configuration for a $moduleUseName service client.\n") customizations.forEach { it.section(ServiceConfig.ConfigStructAdditionalDocs)(writer) @@ -424,9 +431,9 @@ class ServiceConfigGenerator( } writer.rustBlock("impl Config") { + writer.docs(configDocs) writer.rustTemplate( """ - /// Constructs a config builder. pub fn builder() -> Builder { Builder::default() } """, ) @@ -483,7 +490,7 @@ class ServiceConfigGenerator( } writer.rustBlock("impl Builder") { - writer.docs("Constructs a config builder.") + writer.docs(configDocs) writer.rust("pub fn new() -> Self { Self::default() }") builderFromConfigBag()(this)