@AntonKhorev commented on this pull request.
> @@ -1,6 +1,7 @@
<%= tag.head :data => application_data do %>
<%= render :partial => "layouts/meta" %>
<%= javascript_include_tag "turbo", :type => "module" %>
+ <%= javascript_include_tag "make-plural/cardinals" %>
Name collisions. They have `const a`, `const b` etc defined in the global
scope. When I added `make-plural/cardinals` as `//= require ...` in
`application.js` it was picking wrong pluralization rules.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5654#discussion_r1976438665
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5654/review/2652499...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev