From 013a58912b2581c2c3e89e8181fcddc4cdd2c1e8 Mon Sep 17 00:00:00 2001 From: masato-bkn Date: Mon, 20 Nov 2023 17:20:34 +0900 Subject: [PATCH] Remove NOTE tag from `Rails/RedundantActiveRecordAllMethod` description for readability The inline code written in the NOTE section is not being displayed in the RuboCop documentation, making it difficult to read. --- lib/rubocop/cop/rails/redundant_active_record_all_method.rb | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/rubocop/cop/rails/redundant_active_record_all_method.rb b/lib/rubocop/cop/rails/redundant_active_record_all_method.rb index 581efd01e4..ee5fa6afd6 100644 --- a/lib/rubocop/cop/rails/redundant_active_record_all_method.rb +++ b/lib/rubocop/cop/rails/redundant_active_record_all_method.rb @@ -34,8 +34,7 @@ def allowed_receivers # Detect redundant `all` used as a receiver for Active Record query methods. # - # NOTE: For the methods `delete_all` and `destroy_all`, - # this cop will only check cases where the receiver is a model. + # For the methods `delete_all` and `destroy_all`, this cop will only check cases where the receiver is a model. # It will ignore cases where the receiver is an association (e.g., `user.articles.all.delete_all`). # This is because omitting `all` from an association changes the methods # from `ActiveRecord::Relation` to `ActiveRecord::Associations::CollectionProxy`,