ruby-grape / grape / 26340653868

97%

master: 97%

Pull #2742

github

Prune unused requires in lib/grape.rb

Three removals (verified via grep across `lib/` + full RSpec run):

- `active_support/version` — redundant; `require 'active_support'`
  itself loads the version file first.
- `pathname` — no `Pathname` reference anywhere in the gem.

One narrow:

- `active_support/inflector` → `active_support/core_ext/string/inflections`
  — only the String inflector extensions (`demodulize`, `underscore`) are
  used (in `validations.rb` and `util/registry.rb`); the rest of the
  Inflector class/rule machinery isn't reached.

Kept with a comment after a false start during the audit:

- `active_support/core_ext/hash/indifferent_access` — looked unused
  (we only construct `ActiveSupport::HashWithIndifferentAccess.new(...)`
  directly, never call `Hash#with_indifferent_access`), but
  `HashWithIndifferentAccess.new(hash)` internally calls
  `nested_under_indifferent_access` on its input, which is added by
  this core_ext. Dropping it broke ~200 specs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

1088 of 1173 branches covered (92.75%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

3530 of 3589 relevant lines covered (98.36%)

31668.72 hits per line