Skip to content

Commit

Permalink
Remove NOTE tag from Rails/RedundantActiveRecordAllMethod descripti…
Browse files Browse the repository at this point in the history
…on for readability

The inline code written in the NOTE section is not being displayed in the RuboCop documentation, making it difficult to read.
  • Loading branch information
masato-bkn committed Nov 20, 2023
1 parent 8015fc2 commit 013a589
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions lib/rubocop/cop/rails/redundant_active_record_all_method.rb
Original file line number Diff line number Diff line change
Expand Up @@ -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`,
Expand Down

0 comments on commit 013a589

Please sign in to comment.