Skip to content
New issue

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

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

Already on GitHub? # to your account

Support Laravel 12 #361

Closed
wants to merge 2 commits into from
Closed

Conversation

ahmedbally
Copy link

@ahmedbally ahmedbally commented Feb 25, 2025

  • Added or updated tests
  • Added or updated the README.md
  • Detailed changes in the CHANGELOG.md unreleased section

Related Issue/Intent

Changes

Breaking changes

@BenSampo
Copy link
Owner

I don't have a problem with updating support for this package, however I strongly suggest that you migrate away from this package if you are using a modern version of Laravel/PHP and use native enums.

@BenSampo
Copy link
Owner

Look like there are some issues to address with return types. Can you take a look @ahmedbally ?

@ahmedbally
Copy link
Author

sure on it

@ahmedbally
Copy link
Author

fixed please check

spawnia added a commit that referenced this pull request Feb 25, 2025
@spawnia spawnia mentioned this pull request Feb 25, 2025
3 tasks
@spawnia
Copy link
Collaborator

spawnia commented Feb 25, 2025

These updates are often quite tricky. I am giving it a shot with #363 too.

@spawnia
Copy link
Collaborator

spawnia commented Feb 25, 2025

Fixed with #363.

@spawnia spawnia closed this Feb 25, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants