search

beyondcode / laravel-mailbox

Catch incoming emails in your Laravel application
https://beyondco.de/docs/laravel-mailbox/getting-started/introduction
MIT License
1.04k stars 127 forks source link

Add PHP 8.0 support #80

Closed clementmas closed 3 years ago

clementmas commented 3 years ago

Move composer dependency zendframework/zend-mail to laminas/laminas-mail due to Zend Framework migration.

Upgraded the tests to use the new Laminas namespace.

This will help to speed things up when they add PHP 8 support (see https://github.com/laminas/laminas-mail/pull/117) to fix issue https://github.com/beyondcode/laravel-mailbox/issues/78 ;)

Edit, March 9 2021: now that Laminas added support for PHP 8, I updated the composer.json php versions and renamed the PR from "Migrate composer dependency from Zend to Laminas" to "Add PHP 8.0 support"

clementmas commented 3 years ago

Hi @beyondcode, Laminas added support for PHP 8 (see https://github.com/laminas/laminas-mail/pull/117)

I think laravel-mailbox can now be updated to support PHP 8 as well. What do you think?

taqie commented 3 years ago

When can expect php 8 support ?

colinmackinlay commented 3 years ago

Would be great if this PR #80 upgrade to PHP8 could got through @mpociot

elbaylot commented 3 years ago

Great job ! Is it ok @mpociot ?

Spriithy commented 3 years ago

Hey guys, my team really needs to be able to push this to production. Could you merge & release please ? @mpociot

clementmas commented 3 years ago

Hi @mpociot, did you get a chance to take a look at this PR?

ernst77 commented 3 years ago

Seems like there should be a different maintainer...