laminas / laminas-authentication

provides an API for authentication and includes concrete authentication adapters for common use case scenarios
https://docs.laminas.dev/laminas-authentication/
BSD 3-Clause "New" or "Revised" License
24 stars 16 forks source link

Add PHP 8.0 support #14

Closed bfoosness closed 3 years ago

bfoosness commented 3 years ago
Q A
New Feature yes

Description

Close #13

holtkamp commented 3 years ago

@boesing this PR seems to be complete to satisfy https://github.com/laminas/laminas-authentication/issues/13, does it need additional effort/work?

bfoosness commented 3 years ago

The laminas-db dependency is no longer a blocker, but I see I have a couple of issues to fix on this one now. I'll look at those this morning.

weierophinney commented 3 years ago

I've restored the test case to be equivalent to what it was previously (not that it's terribly useful), and rebased your branch off current 2.8.x (which adds our GHA workflow) - all looks good, so I'm merging!