FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Remove illuminate/container patch scaffolding by TomasVotruba · Pull Request #772 · rectorphp/rector-phpunit · GitHub

Remove illuminate/container patch scaffolding - #772

Merged
TomasVotruba merged 1 commit into
mainfrom
remove-illuminate-container-patch
Aug 23, 2026
Merged

Remove illuminate/container patch scaffolding#772
TomasVotruba merged 1 commit into
mainfrom
remove-illuminate-container-patch

Conversation

Copy link
Copy Markdown
Member

rector/rector-src switched its DI container from illuminate/container to entropy/entropy (rectorphp/rector-src#8362), so the Illuminate container patch it used to declare is gone.

This drops the now-dead patch opt-in that this package carried only to apply that upstream patch:

     "require-dev": {
-        "symplify/vendor-patches": "^11.5",
     },
-    "extra": {
-        "enable-patching": true
-    },
     "config": {
         "allow-plugins": {
-            "cweagans/composer-patches": true,
         }
     }

No patch is declared in this repo and none remains upstream, so the scaffolding is pure dead weight.

Note

Green CI here depends on rectorphp/rector-src#8362 being merged first — this package tracks rector/rector-src: dev-main, which still ships the (now unpatched) Illuminate container until that PR lands.

rector/rector-src switched its DI container from illuminate/container to
entropy/entropy, so the Illuminate container patch it used to declare is gone.
Drop the now-dead patch opt-in here: the symplify/vendor-patches dev dep, the
extra.enable-patching flag, and the cweagans/composer-patches allow-plugin.
TomasVotruba merged commit 0f24ff4 into main Aug 23, 2026
7 checks passed
TomasVotruba deleted the remove-illuminate-container-patch branch August 23, 2026 10:07
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL