Rails/HasManyOrHasOneDependent should accept dependent: nil

[grosser]

We use dependent: nil to mean "I though about dependencies and nothing should be done"
it's also a valid option in rails ... so please don't alert on this.

app/models/user.rb:23:3: C: Rails/HasManyOrHasOneDependent: Specify a :dependent option.
  has_many :jobs, dependent: nil, inverse_of: :user

[andyw8]

Hmm, is this option missing from the docs?

https://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#method-i-has_many

[grosser]

yes rails/rails#35504

…

On Sun, Nov 3, 2019 at 3:43 PM Andy Waite ***@***.***> wrote: Hmm, is this option missing from the docs? https://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#method-i-has_many — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#147>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAACYZ5TM732NHT76DE6LFLQR5H2PANCNFSM4JIM6LWA> .

[andyw8]

Thanks. I think it's probably best to wait for a response from someone on the Rails team before this is added to rubocop-rails.

[grosser]

merged into rails now ... good to go here ?

[andyw8]

👍 PR welcome.