Ignore abstract model classes

Open fatkodima opened this issue 3 years ago • 2 comments

$ rake active_record_doctor

Got this warning, among others:

ApplicationRecord references a non-existent table or view named

Mar 12 '22 15:03 fatkodima

I think those errors were introduced after we removed a check for model.table_name.nil?. We probably should reintroduce that check.

Mar 12 '22 16:03 gregnavis

I think those errors were introduced after we removed a check

Double checked - yes. But I think, this PR should cover that case. Any reason to have a model without table_name, except abstract classes?

Mar 12 '22 17:03 fatkodima

@fatkodima, I'm closing this PR as it seems to be no longer relevant after these changes.

Mar 23 '23 11:03 gregnavis