@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/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
