Skip to content

Remove unnecessary debug status checks

Mark Lapierre requested to merge ml-qa-remove-unnecessary-debug-checks into master

What does this MR do?

Runtime::Logger defaults to reporting error messages, so we don't need to check if QA_DEBUG is true when creating debug log messages (or any other message severity below error). If QA_DEBUG is false (including not set at all) the debug messages won't be logged.

So this MR removes a bunch of if Runtime::Env.debug? statements to avoid confusion.

Does this MR meet the acceptance criteria?

Conformity

Merge request reports