Skip to content

Commit

Permalink
AddDisplayStringLineIndented also needs to do output chaining
Browse files Browse the repository at this point in the history
Summary: Addition of this method was missing the required output chaining, causing missing lines in the human-readable log when both JSON and text output are used in noninteractive runs.

Differential Revision: D54494184

fbshipit-source-id: d0cf499b90fe2008b06f621affe9f6c893e0e8cf
  • Loading branch information
elliekorn authored and facebook-github-bot committed Mar 4, 2024
1 parent afe1033 commit 9ffc5a7
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions CommandInfrastructure/JsonStructuredOutput.cs
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,7 @@ public void AddDisplayStringLineIndented(int indent, string format, params objec
// Do not interpret format string.
CurrentDisplayString?.AppendLine(format);
}
m_chainedStructuredOutput?.AddDisplayStringLineIndented(indent, format, args);
}

public void BeginArray(string key)
Expand Down

0 comments on commit 9ffc5a7

Please sign in to comment.