search

standardrb / standard-rails

A Standard Ruby plugin that configures rubocop-rails
MIT License
163 stars 15 forks source link

disable Rails/HasManyOrHasOneDependent #53

Closed Cohen-Carlisle closed 2 months ago

Cohen-Carlisle commented 2 months ago

they say fortune favors the bold so... here's a PR!

See issue (linked below) for my rationale.

closes #50

searls commented 2 months ago

I agree, there are valid cases where you don't want to do this. I might keep it in if Rails offered a better option than nil for specifying "don't do anything" (like :ignore), but even then I'm :-1: on any rule that'd make some people just add more ceremony for the sake of making a linter happy