Skip to content

Use ReleaseTools.logger in DeploymentUtil

Reuben Pereira requested to merge rp/fix-missing-logger into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Use ReleaseTools.logger in DeploymentUtil module to remove the dependency on SemanticLogger::Loggable. This removes the need to include SemanticLogger::Loggable whenever we include DeploymentUtil.

MergeRequestLeadTime included DeploymentUtil without including SemanticLogger::Loggable. This caused errors in some CI jobs. There was no spec that caught this, so I've added a spec that covers this.

Author Check-list

  • Has documentation been updated?
Edited by Reuben Pereira

Merge request reports