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

Comparable enum #20

Merged
merged 13 commits into from
Jan 12, 2024
Merged

Comparable enum #20

merged 13 commits into from
Jan 12, 2024

Conversation

hungthai1401
Copy link
Contributor

Hi @stancl ,
This PR implements comparable enum such as is, isNot, in and notIn operators.
Thanks for the great package!

check Outdated Show resolved Hide resolved
src/Comparable.php Outdated Show resolved Hide resolved
@indytechcook
Copy link

This would be great! What's blocking the merge and cutting a new release?

@stancl
Copy link
Member

stancl commented Jan 12, 2024

@indytechcook Wasn't sure whether to add all the things included in this PR but I figured I'll just merge this.

Updating some CI stuff now, noticed that tests were failing locally (PHP 8.3) so I added more PHP versions to the CI matrix. Seems like there's been a change in 8.2-8.3 that breaks some of our tests. Will fix that here and then merge this into master.

Also dropped Laravel 9 in this PR. I think I'll tag v1.0.0 after merging this, since the package has been pretty stable so there's no reason to stay on 0.x.

@stancl stancl merged commit f0ea4c3 into archtechx:master Jan 12, 2024
5 checks passed
@hungthai1401 hungthai1401 deleted the comparable branch January 15, 2024 03:53
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.

3 participants