Remove php version from composer

[alexz707]

Changes in this pull request

Resolves #

Additional info

[jdreesen]

Why do you remove it?

[alexz707]

Why do you remove it?

I was instructed to do so because as pimcore/pimcore is required in every bundle it should be maintained there.
Only excpetion would be if a bundle needs to narrow down the version e.g. works only with 8.4 then we should keep it in the bundle itself.

Best Alex

[blankse]

@alexz707 I don't think that's a good idea. What if php version 8.5 is added to pimcore/pimcore, but this bundle is not ready for 8.5 yet? Then 8.3/8.4 cannot simply be added here, because then composer will load the latest version without php restriction.

Each package should define its own dependencies and not refer to dependencies of a dependency, these can change.

[alexz707]

@alexz707 I don't think that's a good idea. What if php version 8.5 is added to pimcore/pimcore, but this bundle is not ready for 8.5 yet? Then 8.3/8.4 cannot simply be added here, because then composer will load the latest version without php restriction.

Each package should define its own dependencies and not refer to dependencies of a dependency, these can change.

did not see your reply before.
Yes I completely agree with you and it needs to be discussed again.
Will take that with me to our steering board meeting tomorrow.

For now I'll re-add it
Best Alex

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud