From a95640b31bbdfa7c47a496653133401b918bc216 Mon Sep 17 00:00:00 2001 From: Pieter Oliver Date: Tue, 8 Jul 2025 17:20:53 +0100 Subject: [PATCH] AutoFix RuboCop Style/MultilineIfModifier --- .rubocop_todo.yml | 6 ------ lib/grape_logging/middleware/request_logger.rb | 10 ++++++---- 2 files changed, 6 insertions(+), 10 deletions(-) diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index 6521c68..a4e5034 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -180,12 +180,6 @@ Style/ModuleFunction: Exclude: - 'lib/grape_logging/timings.rb' -# Offense count: 1 -# This cop supports safe autocorrection (--autocorrect). -Style/MultilineIfModifier: - Exclude: - - 'lib/grape_logging/middleware/request_logger.rb' - # Offense count: 1 # This cop supports unsafe autocorrection (--autocorrect-all). # Configuration parameters: EnforcedStyle. diff --git a/lib/grape_logging/middleware/request_logger.rb b/lib/grape_logging/middleware/request_logger.rb index b6e112f..b2fc37e 100644 --- a/lib/grape_logging/middleware/request_logger.rb +++ b/lib/grape_logging/middleware/request_logger.rb @@ -3,10 +3,12 @@ require 'grape' module GrapeLogging module Middleware class RequestLogger < Grape::Middleware::Base - ActiveSupport::Notifications.subscribe('sql.active_record') do |*args| - event = ActiveSupport::Notifications::Event.new(*args) - GrapeLogging::Timings.append_db_runtime(event) - end if defined?(ActiveRecord) + if defined?(ActiveRecord) + ActiveSupport::Notifications.subscribe('sql.active_record') do |*args| + event = ActiveSupport::Notifications::Event.new(*args) + GrapeLogging::Timings.append_db_runtime(event) + end + end # Persist response status & response (body) # to use int in parameters