[openstreetmap/openstreetmap-website] Move spammy phrases to DB (PR #6549)
Tom Hughes
notifications at github.com
Thu Nov 27 16:53:57 UTC 2025
@tomhughes commented on this pull request.
I don't see any major issues here but I am curious about the decision to use classes for the scorers given that we don't actually create objects with any sort of lifetime that need to hold state - we only ever create them long enough to call `spam_score` and then discard them.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6549#pullrequestreview-3516392100
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6549/review/3516392100 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251127/d6ace7f9/attachment.htm>
More information about the rails-dev
mailing list