[openstreetmap/openstreetmap-website] Move `gravatar_enable` method to User model (PR #5988)

[Tom Hughes]

tomhughes left a comment (openstreetmap/openstreetmap-website#5988)

Please just update rubocop_todo for the purposes of this PR and if you want to try and address those metrics warnings then we can do so in separate PRs.

I'm somewhat dubious about some of the class length metrics personally - sure you can "solve" them by moving methods to a concern but a concern that is only used by one class seems a but pointless and to my mind just makes things harder as you have to go hunting around to find out where something is when you want to work on it. I understand that others may disagree however ;-)

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5988#issuecomment-2867093440
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/5988/c2867093440 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250509/1c2a904c/attachment.htm>