ruby-grape / grape
97%
master: 97%

Repo Added 04 Apr 2013 06:08AM UTC

Committed 08 May 2026 06:02PM UTC coverage: 96.921% (-0.01%) from 96.935%

Committed by

Commit Message

Replace .tap usages in lib/ with explicit local variables

`.tap` is idiomatic when you want to perform side effects on a value
and return the value unchanged, but most uses in lib/ were just a
ceremony for "build up a fresh value, then return it" — clearer as
plain locals.

Refactors across 11 files:
- `[].tap { |parts| ... }` → local + explicit return
- `Class.new.tap { |instance| ... }` → local + explicit return
- `Strategies[type].tap { raise unless ... }` → assign-then-raise
- `Rack::Builder.new.tap { ... }` → local + explicit return
- `self.class.new.tap { |s| ... }` → local + explicit return
- `keys.tap { concat + uniq! }` → `(a + b).uniq` with guard clause
- `{}.tap { |details| ... }` → local + explicit return
- `Array.wrap(x).tap { ... }.join(...)` → local, then join
- `Module.new.tap { |mod| ... }` → local + explicit return
- `||= …new.tap { … }` memoization → early-return memoization

Drive-by: replace `handler.instance_of?(Symbol)` with
`handler.is_a?(Symbol)` in `Middleware::Error#run_rescue_handler`
for consistency with the rest of the file (Symbol can't be
subclassed, so the two are behaviorally equivalent here).

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

Coverage Stats

1084 of 1169 branches covered (92.73%)

Branch coverage included in aggregate %.

64 of 68 new or added lines in 11 files covered. (94.12%)

3418 of 3476 relevant lines covered (98.33%)

32141.5 hits per line

3476 RELEVANT LINES 3418 COVERED LINES

32141.5 HITS PER LINE

Builds	Branch	Commit	Type	Ran	Committer	Via	Coverage
25571296655	refactor/replace-tap-usages	Replace .tap usages in lib/ with explicit local variables `.tap` is idiomatic when you want to perform side effects on a value and return the value unchanged, but most uses in lib/ were just a ceremony for "build up a fresh value, then return it"...	Pull #2701	08 May 2026 06:03PM UTC	ericproulx	github	96.92
25554369412	refactor/replace-tap-usages	Replace .tap usages in lib/ with explicit local variables `.tap` is idiomatic when you want to perform side effects on a value and return the value unchanged, but most uses in lib/ were just a ceremony for "build up a fresh value, then return it"...	Pull #2701	08 May 2026 12:03PM UTC	ericproulx	github	96.92
25554351039	refactor/replace-tap-usages	Replace .tap usages in lib/ with explicit local variables `.tap` is idiomatic when you want to perform side effects on a value and return the value unchanged, but most uses in lib/ were just a ceremony for "build up a fresh value, then return it"...	push	08 May 2026 12:00PM UTC	ericproulx	github	96.92

ruby-grape / grape
97%
master: 97%

ruby-grape / grape 97% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

CHANGE BRANCH x

Relevant lines Covered

Source Files on refactor/replace-tap-usages

Recent builds

ruby-grape / grape
97%
master: 97%

README BADGES
x

CHANGE BRANCH
x