Skip to content

Expose public schema reader on ContractScopeValidator#2680

Merged
ericproulx merged 1 commit intomasterfrom
contract_scope_validator_public_schema
Apr 16, 2026
Merged

Expose public schema reader on ContractScopeValidator#2680
ericproulx merged 1 commit intomasterfrom
contract_scope_validator_public_schema

Conversation

@ericproulx
Copy link
Copy Markdown
Contributor

Summary

  • Adds attr_reader :schema to ContractScopeValidator
  • Switches validate method from @schema ivar access to schema reader
  • Consistent with the frozen validator design pattern used across the codebase

Test plan

  • Existing ContractScopeValidator specs pass
  • bundle exec rspec spec/grape/validations/validators/contract_scope_validator_spec.rb

🤖 Generated with Claude Code

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 16, 2026

Danger Report

No issues found.

View run

@ericproulx ericproulx force-pushed the contract_scope_validator_public_schema branch from bfae023 to bf958bf Compare April 16, 2026 10:49
@ericproulx ericproulx force-pushed the contract_scope_validator_public_schema branch from bf958bf to 02a90a2 Compare April 16, 2026 10:55
@ericproulx ericproulx merged commit 15e2d59 into master Apr 16, 2026
79 checks passed
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