-
Notifications
You must be signed in to change notification settings - Fork 47
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
Compatibility issue with Magento #221
Comments
Would you mind sending a PR after confirming it works for you? |
Yes I'll try that :) |
ah true, we had to add a hack for magentos classloader: discovery/src/ClassDiscovery.php Line 244 in 5cc4283
|
1 task
should be fixed with https://github.com/php-http/discovery/releases/tag/1.15.3 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@nicolas-grekas it seems recent changes reintroduced this issue with Magento:
Could be related to this path you added?
\Phalcon\Http\Message\RequestFactory::class
Maybe it would be better to use a string there instead of the ::class syntax?
File
src/Composer/Plugin.php
line 239:Thanks in advance!
The text was updated successfully, but these errors were encountered: