ruby-grape / grape / 24756277250

97%

master: 97%

Pull #2695

github

Lift trailing if/else into guard clauses

Four methods ended in an `if/else` where one branch was a one-line
special-case detour and the other was the main flow. The `else`
existed only as a syntactic counterpart to `if`, pushing multi-line
main paths down an extra indentation level.

- `Grape::DSL::Parameters#requires` — `if opts[:using]` is the
  shortcut form; the main flow is `validate_attributes` + scope/push.
- `Grape::DSL::Parameters#optional` — same shape.
- `Grape::DSL::Parameters#declared_param?` — `if lateral?` delegates
  to `@parent`; the main flow is the `.any?` check.
- `Grape::API.configure` — `unless block_given?` returns the config
  reader; the main flow yields and returns `self`.

Each now uses a `return … if …` guard, leaving the main flow at the
method's top indentation. No behavior change; all 2,236 specs pass.

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

1079 of 1163 branches covered (92.78%)

Branch coverage included in aggregate %.

12 of 12 new or added lines in 2 files covered. (100.0%)

3353 of 3407 relevant lines covered (98.42%)

32199.04 hits per line