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

Update Laravel integration #9

Closed
wants to merge 16 commits into from
Closed

Conversation

vinkla
Copy link
Contributor

@vinkla vinkla commented Aug 12, 2024

This pull request updates the Laravel integration and can be merged once issue #8 is resolved.

  • Added type declarations
  • Simplified the service provider
  • Suggested packages for Laravel
  • Ran tests against Laravel 11
  • Dropped support for Laravel 9 and 10
  • Updated the config file location

Normalizing the composer.json file using the ergebnis/composer-normalize package is another idea for the future.

@olssonm olssonm closed this Aug 12, 2024
@olssonm olssonm reopened this Aug 12, 2024
@olssonm
Copy link
Owner

olssonm commented Aug 12, 2024

Buttery fingers... Sorry browsed the wrong PR, accidentally closed. Reopened again.

@vinkla vinkla mentioned this pull request Aug 12, 2024
@vinkla
Copy link
Contributor Author

vinkla commented Aug 12, 2024

Work continued in #12

@vinkla vinkla closed this Aug 12, 2024
@vinkla vinkla deleted the fix/laravel branch August 12, 2024 17:29
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