gitlab-org--gitlab-foss/rubocop
Sean McGivern 4ef3e3491e Add cop for has_many :through without disabled autoloading
Goldiloader is great, but has several issues with has_many :through relations:

* https://github.com/salsify/goldiloader/issues/12
* https://github.com/salsify/goldiloader/issues/14
* https://github.com/salsify/goldiloader/issues/18

Rather than try to figure out which applies in each case, we should just do the
drudge work of manually disabling autoloading for all relations of this type. We
can always use regular preloading for specific cases, but this way we avoid
generating invalid queries through Goldiloader's magic.
2018-04-09 12:47:04 +01:00
..
cop Add cop for has_many :through without disabled autoloading 2018-04-09 12:47:04 +01:00
migration_helpers.rb Add cop for use of remove_column 2017-12-11 16:34:51 +00:00
rubocop.rb Add cop for has_many :through without disabled autoloading 2018-04-09 12:47:04 +01:00
spec_helpers.rb Introduce a new FactoriesInMigrationSpecs cop 2018-04-06 17:41:52 +02:00