Skip to content

Fixes formatter to allow external use

Richard Chong requested to merge cd8497-logger-fix into main

What does this MR do and why?

Few more tweaks to make this usable.

  • Noticed a bug with the notification in the formatter not being referenced properly.
  • Updates activesupport to ~> 7.1 since that's the minimum needed to use active_support/broadcast_logger which is used in runtime/logger.rb
  • Moves influxdb-client as a runtime_dependency

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Richard Chong

Merge request reports