Skip to content

Commit 9675bcd

Browse files
authored
fix(appconfig): deprecate deploy method (#29021)
### Reason for this change Want to recommend against creating AppConfig resources across multiple stacks. ### Description of changes ### Description of how you validated changes ### Checklist - [ ] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 0efc6ae commit 9675bcd

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

packages/@aws-cdk/aws-appconfig-alpha/lib/configuration.ts

+3
Original file line numberDiff line numberDiff line change
@@ -307,6 +307,9 @@ abstract class ConfigurationBase extends Construct implements IConfiguration, IE
307307
* Deploys the configuration to the specified environment.
308308
*
309309
* @param environment The environment to deploy the configuration to
310+
* @deprecated Use `deployTo` as a property instead. We do not recommend
311+
* creating resources in multiple stacks. If you want to do this still,
312+
* please take a look into https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_appconfig.CfnDeployment.html.
310313
*/
311314
public deploy(environment: IEnvironment) {
312315
const logicalId = `Deployment${this.getDeploymentHash(environment)}`;

0 commit comments

Comments
 (0)