Skip to content

Allow templates to be callables#1489

Merged
henriquemoody merged 1 commit intoRespect:mainfrom
henriquemoody:core/assert-callable
Dec 11, 2024
Merged

Allow templates to be callables#1489
henriquemoody merged 1 commit intoRespect:mainfrom
henriquemoody:core/assert-callable

Conversation

@henriquemoody
Copy link
Member

No description provided.

@henriquemoody henriquemoody added this to the 3.0 milestone Dec 11, 2024
@codecov
Copy link

codecov bot commented Dec 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.24%. Comparing base (298b209) to head (4a16ad3).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1489   +/-   ##
=========================================
  Coverage     97.24%   97.24%           
- Complexity      951      952    +1     
=========================================
  Files           198      198           
  Lines          2320     2323    +3     
=========================================
+ Hits           2256     2259    +3     
  Misses           64       64           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@henriquemoody henriquemoody force-pushed the core/assert-callable branch 3 times, most recently from 162264f to 583c475 Compare December 11, 2024 16:46
There are a few use cases in which you would like to have a custom
exception but, at the same time, reuse the message or exception that
validation might give you.

This commit creates a new feature that allows users to define a callable
that will generate an exception when it fails.
@henriquemoody henriquemoody marked this pull request as ready for review December 11, 2024 16:52
@henriquemoody henriquemoody merged commit 4a16ad3 into Respect:main Dec 11, 2024
@henriquemoody henriquemoody deleted the core/assert-callable branch December 11, 2024 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant