Skip to content

v5.1.2 - #89

Merged
enjaku4 merged 12 commits into
mainfrom
v5.1.2
Aug 9, 2025
Merged

v5.1.2#89
enjaku4 merged 12 commits into
mainfrom
v5.1.2

Conversation

@enjaku4

@enjaku4 enjaku4 commented Aug 9, 2025

Copy link
Copy Markdown
Owner

No description provided.

@enjaku4
enjaku4 requested a review from Copilot August 9, 2025 09:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR represents a version release v5.1.2 that focuses on minor performance improvements and code cleanup. The primary changes involve refactoring the input validation system, introducing more specific error types, and relaxing dependency version constraints.

  • Refactored the input validation system from a monolithic Rabarber::Inputs module to individual class-based validators
  • Introduced InvalidContextError as a more specific error type for context validation issues
  • Relaxed dependency versions for dry-configurable and dry-types to support older versions

Reviewed Changes

Copilot reviewed 36 out of 37 changed files in this pull request and generated no comments.

Show a summary per file
File Description
lib/rabarber/inputs.rb Removed monolithic input validation module
lib/rabarber/inputs/*.rb Added individual class-based input validators
lib/rabarber/*.rb Updated to use new input validation classes and specific error types
spec/**/*_spec.rb Updated test expectations to use InvalidContextError and added tests for new input classes
rabarber.gemspec Relaxed dependency version constraints
lib/rabarber/version.rb Bumped version to 5.1.2

@enjaku4
enjaku4 merged commit dfd9dfb into main Aug 9, 2025
15 checks passed
@enjaku4
enjaku4 deleted the v5.1.2 branch August 9, 2025 09:49
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.

2 participants