Sacrifice for the linting gods

DataDog · Apr 27, 2022 · 2294299 · 2294299
1 parent ae2fef3
commit 2294299
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/datadog/tracing/contrib/rails/patcher.rb b/lib/datadog/tracing/contrib/rails/patcher.rb
@@ -83,15 +83,17 @@ def add_logger(app)
             # if so, add proc that injects trace identifiers for tagged logging.
             logger = app.config.logger || ::Rails.logger
 
-            if (logger) \
+            if logger \
                 && defined?(::ActiveSupport::TaggedLogging) \
                 && logger.is_a?(::ActiveSupport::TaggedLogging)
 
               Contrib::Rails::LogInjection.add_as_tagged_logging_logger(app)
               should_warn = false
             end
 
-            Datadog.logger.warn("Unable to enable Datadog Trace context, Logger #{logger.class} is not supported") if should_warn
+            if should_warn
+              Datadog.logger.warn("Unable to enable Datadog Trace context, Logger #{logger.class} is not supported")
+            end
           end
 
           def patch_after_intialize