[openstreetmap/openstreetmap-website] Add URL prettification for plain texts (PR #6518)

Marwin Hochfelsner notifications at github.com
Thu Dec 4 16:03:27 UTC 2025


@hlfan commented on this pull request.



> +      Array.wrap(Settings.linkify&.detection_rules)
+           .select { |rule| rule.path_template && rule.patterns.is_a?(Array) }
+           .flat_map { |rule| linkify_detection_rule_to_gsub(rule) }

I thought about that too when I broke the flat_mapping out into `linkify_detection_rule_to_gsub(rule)` to make RuboCop happy. What issues were you thinking about for memoisation?

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

Message ID: <openstreetmap/openstreetmap-website/pull/6518/review/3540765053 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20251204/64490db3/attachment.htm>


More information about the rails-dev mailing list