[openstreetmap/openstreetmap-website] Add eslint-plugin-erb to provide linting of .js.erb files (PR #5559)

Marwin Hochfelsner notifications at github.com
Tue Feb 4 16:31:23 UTC 2025


@hlfan commented on this pull request.



> +  MAX_REQUEST_AREA: <%= Settings.max_request_area.to_json %>,
+  SERVER_PROTOCOL: <%= Settings.server_protocol.to_json %>,
+  SERVER_URL: <%= Settings.server_url.to_json %>,
+  API_VERSION: <%= Settings.api_version.to_json %>,
+  STATUS: <%= Settings.status.to_json %>,
+  MAX_NOTE_REQUEST_AREA: <%= Settings.max_note_request_area.to_json %>,
+  OVERPASS_URL: <%= Settings.overpass_url.to_json %>,
+  OVERPASS_CREDENTIALS: <%= Settings.overpass_credentials.to_json %>,
+  NOMINATIM_URL: <%= Settings.nominatim_url.to_json %>,
+  GRAPHHOPPER_URL: <%= Settings.graphhopper_url.to_json %>,
+  FOSSGIS_OSRM_URL: <%= Settings.fossgis_osrm_url.to_json %>,
+  FOSSGIS_VALHALLA_URL: <%= Settings.fossgis_valhalla_url.to_json %>,

There is the `key-spacing: ["error", { align: "value" }]` linting rule available for objects too, but moving this to Ruby seems like the right thing to do.

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

Message ID: <openstreetmap/openstreetmap-website/pull/5559/review/2593427454 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250204/09ed74ed/attachment-0001.htm>


More information about the rails-dev mailing list