Skip to content

Commit

Permalink
fix(cli): change-set arn is not printed on cdk deploy --no-execute (#…
Browse files Browse the repository at this point in the history
…166)

Fixes: #160

Fixes three message statements that did not get send to the IoHost.

---
By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache-2.0 license
  • Loading branch information
mrgrain authored Feb 27, 2025
1 parent f925551 commit bf33829
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions packages/aws-cdk/lib/api/deployments/deploy-stack.ts
Original file line number Diff line number Diff line change
Expand Up @@ -426,9 +426,9 @@ class FullCloudFormationDeployment {
await this.updateTerminationProtection();

if (changeSetHasNoChanges(changeSetDescription)) {
debug(this.action, format('No changes are to be performed on %s.', this.stackName));
await this.ioHost.notify(debug(this.action, format('No changes are to be performed on %s.', this.stackName)));
if (execute) {
debug(this.action, format('Deleting empty change set %s', changeSetDescription.ChangeSetId));
await this.ioHost.notify(debug(this.action, format('Deleting empty change set %s', changeSetDescription.ChangeSetId)));
await this.cfn.deleteChangeSet({
StackName: this.stackName,
ChangeSetName: changeSetName,
Expand Down Expand Up @@ -457,10 +457,10 @@ class FullCloudFormationDeployment {
}

if (!execute) {
info(this.action, format(
await this.ioHost.notify(info(this.action, format(
'Changeset %s created and waiting in review for manual execution (--no-execute)',
changeSetDescription.ChangeSetId,
));
)));
return {
type: 'did-deploy-stack',
noOp: false,
Expand Down

0 comments on commit bf33829

Please sign in to comment.