Breaking: Add support for laravel 11, test on php 8.3, remove old laravel versions

ssi-anik / laravel-amqp

anik/amqp wrapper for Laravel-ish frameworks

https://packagist.org/packages/anik/laravel-amqp

MIT License

31 stars 4 forks source link

Breaking: Add support for laravel 11, test on php 8.3, remove old laravel versions #11

Closed steache closed 6 months ago

ssi-anik commented 6 months ago

@steache I am sorry for replying this late.

I was thinking of dropping L6,7,8,9 and only keeping L10 and onwards with PHP ^8.1

Can you make those changes on the composer as well as the workflow yml files?

TIA.

EDIT: also in the readme, in case someone wants to use the removed versions, they get forwarded to the older versions.

steache commented 6 months ago

@ssi-anik so something like this?
I've removed old versions
I did not get the part with readme. :)
I've changed the title, becase this will be breaking change, not fix.

codecov-commenter commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.31%. Comparing base (8759b81) to head (ff53b72).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #11 +/- ## ========================================= Coverage 99.31% 99.31% Complexity 84 84 ========================================= Files 6 6 Lines 292 292 ========================================= Hits 290 290 Misses 2 2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

ssi-anik commented 6 months ago

@steache thanks mate.

Those Laravel versions are already deprecated and there is no code-related change so far. So I dropped all those previous versions. And, I am going to make a new tag so the upcoming changes (if any) should go under new tags.

Finally, thanks for the help. 🙏

ssi-anik commented 6 months ago

@steache I have added new tag v2.0.