Fixes from review comments

pulumi · Apr 30, 2024 · 6c2125f · 6c2125f
1 parent bb929a1
commit 6c2125f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/...pect-no-changes-now-causes-pulumi-cli-to-fail-if-the-only-changes-are-output-changes.yaml b/...pect-no-changes-now-causes-pulumi-cli-to-fail-if-the-only-changes-are-output-changes.yaml
@@ -1,4 +1,4 @@
 changes:
 - type: fix
  scope: cli/engine
- description: "Fixes #15564, --expect-no-changes now causes pulumi cli to fail if the only changes are output changes"
+ description: "Make --expect-no-changes fail even if the only changes are output changes"
diff --git a/pkg/cmd/pulumi/stack_history.go b/pkg/cmd/pulumi/stack_history.go
@@ -163,6 +163,8 @@ func displayUpdatesJSON(updates []backend.UpdateInfo, decrypter config.Decrypter
  info.EndTime = makeStringRef(time.Unix(update.EndTime, 0).UTC().Format(timeFormat))
  resourceChanges := make(map[string]int)
  for k, v := range update.ResourceChanges {
+ // Filter out the the OpOutputChange events because they are pseudo
+ // events that shouldn't be included in the stack history
  if k != deploy.OpOutputChange {
  resourceChanges[string(k)] = v
  }