ruby-grape / grape / 25137726391 / 5

97%

master: 97%

push

github

Lift trailing if/else into guard clauses (#2695)

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>

1053 of 1163 branches covered (90.54%)

Branch coverage included in aggregate %.

3308 of 3397 relevant lines covered (97.38%)

1009.86 hits per line