[openstreetmap/openstreetmap-website] Remove json dependency (PR #6348)
rkoeze
notifications at github.com
Sun Aug 24 01:45:19 UTC 2025
### Description
Ruby has shipped with the default JSON gem since 1.9, so there's no need to require this in the Gemfile unless we want to pin a version (which I don't think we do).
I don't think `multi_json` requires us to explicitly require the gem either. They mention in their docs [the JSON gem that ships with Ruby is supported](https://github.com/sferik/multi_json?tab=readme-ov-file#supported-json-engines). I also don't see anything concerning in the `multi_json` [adapter](https://github.com/sferik/multi_json/blob/main/lib/multi_json/adapters/json_gem.rb#L7).
### How has this been tested?
The tests pass.
You can view, comment on, or merge this pull request online at:
https://github.com/openstreetmap/openstreetmap-website/pull/6348
-- Commit Summary --
* Remove json dependency
-- File Changes --
M Gemfile (3)
M Gemfile.lock (1)
-- Patch Links --
https://github.com/openstreetmap/openstreetmap-website/pull/6348.patch
https://github.com/openstreetmap/openstreetmap-website/pull/6348.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6348
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6348 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250823/8396fd00/attachment.htm>
More information about the rails-dev
mailing list