-
Notifications
You must be signed in to change notification settings - Fork 211
Autoloader: Implementing composer autoloader with classmap #4386
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
Autoloader: Implementing composer autoloader with classmap #4386
Conversation
📈 PHP Unit Code Coverage Report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wjrosa, can you change the target of this PR to the try/add-basic-classmap-autoloader
branch? The current target branch is dev/renaming-the-main-extension-files
, which includes PSR-4 class renames, which aren't necessary for the classmap to function.
91e95eb
to
f740537
Compare
@daledupreez I messed it up with the rebase, so I created another one: #4406 |
Fixes STRIPE-507
See peNR48-1sM-p2#comment-879
Changes proposed in this Pull Request:
As part of the autoloader implementation, this PR changes the existing version to use Composer's classmap version instead. This should provide Composer with useful features such as the auto-generation of the classmap file and cache.
Testing instructions
Install
dev/implementing-psr-4-autoloader-without-namespaces
)npm run build:deps
Build
npm run build
vendor
folderChangelog entry
Changelog Entry Comment
Comment
Post merge