-
Notifications
You must be signed in to change notification settings - Fork 25
Laravel 5.3 Support #16
Comments
Is anyone working on this? |
Is this package dead? |
It would be great to know what is going on with this project.... |
There is already closed issue #15 But it seems nobody cares. |
I'll make a pull request soon, it is working for me locally. |
Ufff, that would be great! Or if you make your repo public :) |
The reason I don't make a PR is because some tests fail. You can check the code in my fork https://github.com/drakakisgeo/iron-queue and the changes here -> https://github.com/LaravelCollective/iron-queue/compare/5.2...drakakisgeo:dev-5.3?expand=1 |
Size method using wrong code copied from SQS queue implementation: Use this instead: public function size($queue = null)
{
return 0;
} |
Replaced that @majksner , thanks! |
It would just be great if we could have some more contributors authorized to approve PR's etc to keep this going. Iron.io and Push queues are very under-appreciated. I would hate to see this package continue to remain unmaintained. |
It seems like original authors/maintainers ditched this package. I've decided to fork it and maintain it on my own. I've also added L5.3 support. You can find it here: https://github.com/HydrefLab/iron-queue Feel free to use it. All contributions are most welcome :) |
The L5.3 support has been merged into the new branch 5.3 and is now tagged for composer |
Hey!
Will it be possible to use this package with laravel 5.3? If not I have to find another solution.
Thank you!
Best regards!
The text was updated successfully, but these errors were encountered: