Skip to content

Commit

Permalink
Merge pull request #24 from r7kamura/remove-column-cache-note
Browse files Browse the repository at this point in the history
Add comment about false-negatives on cached Migration/RemoveColumn
  • Loading branch information
r7kamura authored Nov 27, 2024
2 parents 9215a3f + 5ee801a commit 22007d7
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions lib/rubocop/cop/migration/remove_column.rb
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,9 @@ module Migration
#
# Active Record caches database columns at runtime, so if you drop a column, it can cause exceptions until your app reboots.
#
# Note that since this cop goes to read arbitrary model files,
# false negatives occur in situations where RuboCop's cache is being used.
#
# @safety
# The logic to check if it is included in `ignored_columns` may fail.
#
Expand Down

0 comments on commit 22007d7

Please sign in to comment.