Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Validation Rule Contract in Rule rule #362

Merged
merged 1 commit into from
Mar 10, 2023

Conversation

EriBloo
Copy link
Contributor

@EriBloo EriBloo commented Feb 27, 2023

Added support for Illuminate\Contracts\Validation\ValidationRule in Spatie\LaravelData\Attributes\Validation\Rule. Didn't remove depreciated contracts for compability reasons.

@rubenvanassche
Copy link
Member

Looks like some tests are failing

@EriBloo
Copy link
Contributor Author

EriBloo commented Mar 2, 2023

Yes. Those failing tests are not related to my pull. After reverting my commit:

image

tests are still failing:

image

it seems they are related to typescript tranformer:

image

It would be weird if my commit, which only adds additional type, was breaking it.

@EriBloo
Copy link
Contributor Author

EriBloo commented Mar 3, 2023

Ok I can see now that it fails in Laravel 9. That's to be expected since new Validation Rule didn't exist back then. Problem is that Laravel Data should have a new major version for Laravel 10 - the way it is right now either Laravel 9 or Laravel 10 is going to fail.

@rubenvanassche
Copy link
Member

Thanks!

@rubenvanassche rubenvanassche merged commit 197e050 into spatie:main Mar 10, 2023
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