From 4033928b0c6b5b06d22bd13cdc81382b8494c105 Mon Sep 17 00:00:00 2001 From: Christian Sutter Date: Fri, 8 Dec 2023 11:48:54 +0000 Subject: [PATCH] Remove `oj` gem Other GOV.UK teams have moved away from using this because it doesn't provide enough of a performance benefit to be worth the extra dependency and upgrade hassles. --- Gemfile | 1 - Gemfile.lock | 2 -- config/initializers/json.rb | 3 --- 3 files changed, 6 deletions(-) delete mode 100644 config/initializers/json.rb diff --git a/Gemfile b/Gemfile index 3e412f6..c907443 100644 --- a/Gemfile +++ b/Gemfile @@ -15,7 +15,6 @@ gem "govuk_app_config" gem "govuk_message_queue_consumer" gem "jsonpath" gem "loofah" -gem "oj" group :test do gem "grpc_mock" diff --git a/Gemfile.lock b/Gemfile.lock index 6a7483e..5a29206 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -166,7 +166,6 @@ GEM racc (~> 1.4) nokogiri (1.15.5-x86_64-linux) racc (~> 1.4) - oj (3.16.1) opentelemetry-api (1.2.3) opentelemetry-common (0.20.0) opentelemetry-api (~> 1.0) @@ -520,7 +519,6 @@ DEPENDENCIES json_schemer jsonpath loofah - oj railties (= 7.0.7.2) rspec-rails rubocop-govuk diff --git a/config/initializers/json.rb b/config/initializers/json.rb deleted file mode 100644 index e1f6e94..0000000 --- a/config/initializers/json.rb +++ /dev/null @@ -1,3 +0,0 @@ -# Use Oj for faster JSON handling across the app -Oj.default_options = { mode: :rails } -Oj.optimize_rails