From 86f2485b2d74b3f046104466edd25bad9f1a5c7b Mon Sep 17 00:00:00 2001 From: Pieter Oliver Date: Mon, 30 Jun 2025 10:50:37 +0100 Subject: [PATCH] AutoFix RuboCop Style/ZeroLengthPredicate --- .rubocop_todo.yml | 5 ----- lib/grape_logging/formatters/rails.rb | 2 +- 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index b554271..28d0ab1 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -457,11 +457,6 @@ Style/TrailingCommaInHashLiteral: - 'spec/lib/grape_logging/loggers/filter_parameters_spec.rb' - 'spec/lib/grape_logging/loggers/response_spec.rb' -# Offense count: 1 -# This cop supports unsafe autocorrection (--autocorrect-all). -Style/ZeroLengthPredicate: - Exclude: - - 'lib/grape_logging/formatters/rails.rb' # Offense count: 3 # This cop supports safe autocorrection (--autocorrect). diff --git a/lib/grape_logging/formatters/rails.rb b/lib/grape_logging/formatters/rails.rb index 1451862..eb7182f 100644 --- a/lib/grape_logging/formatters/rails.rb +++ b/lib/grape_logging/formatters/rails.rb @@ -47,7 +47,7 @@ module GrapeLogging message << " Parameters: #{params.inspect}\n" if params message << "Completed #{status} #{::Rack::Utils::HTTP_STATUS_CODES[status]} in #{total_time}ms" - message << " (#{additions.join(" | ".freeze)})" if additions.size > 0 + message << " (#{additions.join(" | ".freeze)})" unless additions.empty? message << "\n" message << "\n" if defined?(::Rails.env) && ::Rails.env.development?