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

Composer 2.0 PSR-4 compatability #1

Closed
jamie-accessfertility opened this issue Nov 3, 2020 · 3 comments
Closed

Composer 2.0 PSR-4 compatability #1

jamie-accessfertility opened this issue Nov 3, 2020 · 3 comments
Labels
bug Something isn't working

Comments

@jamie-accessfertility
Copy link
Contributor

Running composer install/update while running composer ^2.0 and package version ^1.0 will cause:

Class JustSteveKing\EloquentLogDriver\Src\Loggers\EloquentLogger located in /vendor/juststeveking/eloquent-log-driver/src\Loggers\EloquentLogger.php does not comply with psr-4 autoloading standard. Skipping.

Updating the namespace within that class from -
namespace JustSteveKing\EloquentLogDriver\Logger;

to

namespace JustSteveKing\EloquentLogDriver\Loggers; will fix the issue.

Thanks

@JustSteveKing
Copy link
Owner

Hey!

I will add this to my list, but if you would like to submit a PR to guarantee it getting solved soon I'm happy to accept one!

@JustSteveKing JustSteveKing added the bug Something isn't working label Nov 3, 2020
@jamie-accessfertility
Copy link
Contributor Author

Thanks JustSteveKing!

Theres another issue when trying to upgrade to Laravel 8 too which I don't mind fixing if that's okay.

Thanks again.

@JustSteveKing
Copy link
Owner

Feel free!!

I actually only built this package originally for the project you're working on ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants