-
Notifications
You must be signed in to change notification settings - Fork 23
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
do not require symfony/symfony to be present in composer.json #39
Comments
instead of check sf/sf composer req, can we test if interface |
Got the same problem with pimcore framework (https://github.com/pimcore/pimcore) it uses composer merger plugin so same problem as above - symfony is not a root dependency in composer.json |
Agreed. It should be fairly easy to look for the required individual Symfony packages in |
Same here. We don't use We should have another way of enabling the extension. |
Has there been any movement on this? I'm trying to use it on a Drupal project where symfony isn't required in the root |
I'm desperate for this in my Drupal 8.x project. |
I wonder if we can leverage |
To be honest, there doesn't seem much desire to provide this. So, unless someone plans to submit a PR, I wouldn't expect the maintainers to get on it. |
We'd need something to "bridge" the gap like https://github.com/Haehnchen/idea-php-drupal-symfony2-bridge does for PHP Storm. I'm working on a fork to parse |
@amaisano any progress on that fork? |
Symfony's default installation through the There needs to be a way to either ignore the requirement or have it fallback to detecting individual packages like symfony/console and symfony/framework-bundle. Edit: Apparently it does check for symfony/framework-bundle. I'm noticing some errors in the extension host log so I am going to investigate those. |
In some Symfony applications, like the Contao 4 Managed Edition, you will never have
symfony/symfony
orsymfony/framework-bundle
as a root dependency (neither of which are required anyway).This extension should not rely on that.
The text was updated successfully, but these errors were encountered: