Skip to content

Internalize contract key maps behind Grape::Util::InheritableSetting accessors#2804

Open
ericproulx wants to merge 1 commit into
masterfrom
inheritable-setting-contract-key-map
Open

Internalize contract key maps behind Grape::Util::InheritableSetting accessors#2804
ericproulx wants to merge 1 commit into
masterfrom
inheritable-setting-contract-key-map

Conversation

@ericproulx

Copy link
Copy Markdown
Contributor

Continues (and completes the key-conversion phase of) the InheritableSetting cleanup from #2795#2803: the Dry::Schema key maps registered by contract blocks are now recorded and read through intention-revealing accessors on Grape::Util::InheritableSetting.

New accessors on InheritableSetting

Accessor Replaces
contract_key_maps / add_contract_key_map(key_map) namespace_stackable[:contract_key_map]

Notes:

🤖 Generated with Claude Code

…accessors

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ericproulx
ericproulx requested a review from dblock July 18, 2026 13:36
@github-actions

Copy link
Copy Markdown

Danger Report

No issues found.

View run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant