Skip to content

Commit

Permalink
Merge pull request #587 from clncy/noexec-changes
Browse files Browse the repository at this point in the history
Remove confirmation prompt for deploy --no-exec
  • Loading branch information
ericzbeard authored Nov 11, 2024
2 parents efb944c + 539dae0 commit 2fe58a0
Showing 1 changed file with 13 additions and 4 deletions.
17 changes: 13 additions & 4 deletions internal/cmd/deploy/deploy.go
Original file line number Diff line number Diff line change
Expand Up @@ -182,6 +182,19 @@ To list and delete changesets, use the ls and rm commands.
}
spinner.Pop()

// Display changeset and exit
if noexec {
spinner.Push("Formatting change set")
status := formatChangeSet(stackName, changeSetName)
spinner.Pop()

fmt.Println("Changeset contains the following changes:")
fmt.Println(status)

fmt.Println("changeset created but not executed:", changeSetName)
return
}

// Confirm changes
if !yes {
spinner.Push("Formatting change set")
Expand All @@ -208,10 +221,6 @@ To list and delete changesets, use the ls and rm commands.
}
}

if noexec {
fmt.Println("changeset created but not executed:", changeSetName)
return
}
}

// Deploy!
Expand Down

0 comments on commit 2fe58a0

Please sign in to comment.