LocalTests were broken in #3575 #5807

Open
opened 2026-01-31 00:22:17 +00:00 by claunia · 0 comments
Owner

Originally created by @zadjii-msft on GitHub (Jan 7, 2020).

Originally assigned to: @zadjii-msft on GitHub.

I'm not sure why this wasn't picked up by the build bot, but this test now fails for me when run locally because of the trailing comma (that's not valid JSON).

4882917499/src/cascadia/LocalTests_TerminalApp/SettingsTests.cpp (L431-L439)

Originally posted by @j4james in https://github.com/microsoft/terminal/pull/3575

Originally created by @zadjii-msft on GitHub (Jan 7, 2020). Originally assigned to: @zadjii-msft on GitHub. I'm not sure why this wasn't picked up by the build bot, but this test now fails for me when run locally because of the trailing comma (that's not valid JSON). https://github.com/microsoft/terminal/blob/488291749994822ef54ba5140c7726b6d984116f/src/cascadia/LocalTests_TerminalApp/SettingsTests.cpp#L431-L439 _Originally posted by @j4james in https://github.com/microsoft/terminal/pull/3575_
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#5807