Skip to content

Commit

Permalink
minor symfony#44765 [ProxyManagerBridge] Remove the unused dependency…
Browse files Browse the repository at this point in the history
… on composer/package-versions-deprecated (stof)

This PR was merged into the 4.4 branch.

Discussion
----------

[ProxyManagerBridge] Remove the unused dependency on composer/package-versions-deprecated

| Q             | A
| ------------- | ---
| Branch?       | 4.4
| Bug fix?      | yes/no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | -
| License       | MIT
| Doc PR        | n/a

The ProxyManagerBridge depends on friendsofphp/proxy-manager-lts which has an optional dependency on the PackageVersions class to implement the `ProxyManager\Version::getVersion` method on composer 1 (with a fallback to a less precise version number).
However, the bridge has stopped using that API in favor of feature detection (in symfony#39017), so the dependency is unused.

This contributes to symfony#44726

Commits
-------

6ca8e30 Remove the unused dependency on composer/package-versions-deprecated
  • Loading branch information
nicolas-grekas committed Dec 22, 2021
2 parents 3526a32 + 6ca8e30 commit 9d9f27a
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion src/Symfony/Bridge/ProxyManager/composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@
],
"require": {
"php": ">=7.1.3",
"composer/package-versions-deprecated": "^1.8",
"friendsofphp/proxy-manager-lts": "^1.0.2",
"symfony/dependency-injection": "^4.0|^5.0",
"symfony/polyfill-php80": "^1.16"
Expand Down

0 comments on commit 9d9f27a

Please sign in to comment.