-
Notifications
You must be signed in to change notification settings - Fork 67
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
Disable module autoload #86
base: main
Are you sure you want to change the base?
Conversation
@rzkhrv can you explain why you would want this? |
Hi @inxilpro, my case is when I remove a module from composer.json I get some errors... |
Oh cool. Can you think of any reason this shouldn't just be the default? I don't know that we even need a config flag… |
I don't have any reason, basically we can remove the config flag |
I agree. Feels like should be the default. I like this change. |
# Conflicts: # config.php # src/Support/AutoDiscoveryHelper.php
Hi @inxilpro, I did it but my IDE doesn't want to format the code, can you help me? |
This change resolves the problem of autoloading files for modules that are not in the composer.json.