ruby-grape / grape
97%
master: 100%

Repo Added 04 Apr 2013 06:08AM UTC

Committed 30 Aug 2026 04:00PM UTC coverage: 96.98% (-0.001%) from 96.981%

Committed by

Commit Message

Name the group-merged options instead of rebinding `opts`

`opts = @group.deep_merge(opts) if @group` never modified what the caller
passed — `deep_merge` returns a new Hash, and the assignment only rebinds a
local — but it left one name meaning two things: the options as declared
before the line, and the options resolved against the enclosing `with`
after it. Every read below the line has to know which side it is on, and
that ambiguity is what put the group type and the presence message on the
wrong side of the merge in the first place.

Give the resolved value its own name, the way `with` already does two
methods up:

    new_group_attrs = @group&.deep_merge(opts) || opts

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>

Coverage Stats

1240 of 1332 branches covered (93.09%)

Branch coverage included in aggregate %.

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

3898 of 3966 relevant lines covered (98.29%)

27174.45 hits per line

3966 RELEVANT LINES 3898 COVERED LINES

27174.45 HITS PER LINE

Builds	Branch	Commit	Type	Ran	Committer	Via	Coverage
33321217439	params-declare-extraction	Name the group-merged options instead of rebinding `opts` `opts = @group.deep_merge(opts) if @group` never modified what the caller passed — `deep_merge` returns a new Hash, and the assignment only rebinds a local — but it left one name meaning t...	Pull #2867	30 Aug 2026 04:00PM UTC	web-flow	github	96.98
33320914238	params-declare-extraction	Name the group-merged options instead of rebinding `opts` `opts = @group.deep_merge(opts) if @group` never modified what the caller passed — `deep_merge` returns a new Hash, and the assignment only rebinds a local — but it left one name meaning t...	Pull #2867	30 Aug 2026 03:55PM UTC	ericproulx	github	96.97
33320360868	params-declare-extraction	Name the group-merged options instead of rebinding `opts` `opts = @group.deep_merge(opts) if @group` never modified what the caller passed — `deep_merge` returns a new Hash, and the assignment only rebinds a local — but it left one name meaning t...	Pull #2867	30 Aug 2026 03:46PM UTC	ericproulx	github	96.97
33320069736	params-declare-extraction	Name the group-merged options instead of rebinding `opts` `opts = @group.deep_merge(opts) if @group` never modified what the caller passed — `deep_merge` returns a new Hash, and the assignment only rebinds a local — but it left one name meaning t...	Pull #2867	30 Aug 2026 03:39PM UTC	ericproulx	github	96.97
33252396426	params-declare-extraction	Name the group-merged options instead of rebinding `opts` `opts = @group.deep_merge(opts) if @group` never modified what the caller passed — `deep_merge` returns a new Hash, and the assignment only rebinds a local — but it left one name meaning t...	Pull #2867	29 Aug 2026 12:24PM UTC	ericproulx	github	96.96
33252280393	params-declare-extraction	Drop `validate_attributes`, which only defaulted a type that never applied `validate_attributes` wrapped a single delegation around one defaulting step: opts = opts.merge(type: Array) if block && opts[:type].nil? validates(attrs, opts, r...	Pull #2867	29 Aug 2026 12:21PM UTC	ericproulx	github	96.96
33251982774	params-declare-extraction	Extract the shared body of `requires` and `optional` With the group-type check unified (#2864), requiredness passed as an argument (#2865) and `as` taken as a keyword (#2866), the two methods differ only in the name they redispatch a legacy Hash ...	Pull #2867	29 Aug 2026 12:14PM UTC	ericproulx	github	96.97

ruby-grape / grape
97%
master: 100%

ruby-grape / grape 97% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

CHANGE BRANCH x

Relevant lines Covered

Source Files on params-declare-extraction

Recent builds

ruby-grape / grape
97%
master: 100%

README BADGES
x

CHANGE BRANCH
x