Skip to content

cleanup: ensure log and output messages have a '\n' terminator

Matt Harbison requested to merge topic/stable/message-newlines into branch/stable

I noticed this in a blackbox.log where the timestamp for the message after obsdiscovery:452 was merged with the previous message.

(I assume the test failures were pre-existing. They don't look like things that would have changed because if this. IDK where the '\n' comes from when printing evolve.obsdiscovery.obsdiscovery_skip_message. It's printed as-is in various ui.status(), but if I append a '\n' to it, some of the tests end up with a blank line in the middle of the output, so I backed that off.)

Merge request reports

Loading