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

remove doctrine #17

Merged

Conversation

macbookandrew
Copy link

Since Laravel 11 removed doctrine/dbal and added first-party methods, this package can work without it as well.

However, since this requires L11, I figured that warrants a 2.0.0 release to avoid breaking older apps.

This also adds a couple other types:

  • array/json
  • guid/uuid
  • binary

@TartanLeGrand
Copy link
Owner

Why not in a world of technology, it will surely encourage people to upgrade their technology 😄.

Same as your other PR, can you make an conventional commit and remove the changes on the changelog the release bot make the release in function of your commit. You have made an breaking change: https://www.conventionalcommits.org/en/v1.0.0/#commit-message-with--to-draw-attention-to-breaking-change

Thank for your contributing. 🙏

@macbookandrew
Copy link
Author

updated

@TartanLeGrand TartanLeGrand merged commit c589173 into TartanLeGrand:main Aug 1, 2024
9 checks passed
TartanLeGrand added a commit that referenced this pull request Aug 1, 2024
TartanLeGrand pushed a commit that referenced this pull request Aug 2, 2024
BREAKING CHANGE: requires Laravel 11
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