Added alternative to composer autoloader #388
Merged
Conversation
This file contains 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
j0k3r
reviewed
Feb 9, 2021
Co-authored-by: Jérémy Benoist <[email protected]>
…er into pr/provide-alt-autoloader
this is weird, because it works on my local machine which also runs PHP 7.4
Running it together with Composer made no sense.
|
Non-Composer-plain-old-manual-include user here... looks good to me. :-) Go for it! |
|
Thank you for the feedback. If problems related to this PR occur, please open a new issue. |
|
I pulled the latest master branch, which included this merge, and tested it on my local machine using the non-Composer (alt_autoload.php-dist) method. It worked great! Thank you @k00ni for your effort on this. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR provides a new file which acts as an alternative autoloader to Composer. It should make manual work obsolete in case Composer is not available. I named the autoloader file
alt_autoload.php-distso people can rename it to e.g.alt_autoload.phpand adapt it as they see fit. This way they can make adaptions without to worry to override them on an update. Also extendedREADME.mdwith a basic installation section.Also added a basic test which checks if this works.
Fixes #117
I would love to hear feedback on this, especially from people who don't use Composer. If this PR is being accepted, we may close #117 and add a reference to the README.md.
CC @apmuthu