### 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/6...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to