bedrock-autoloader icon indicating copy to clipboard operation
bedrock-autoloader copied to clipboard

Published 20 hours ago • verified publisher icon roots

Change namespace?

Open austinpray opened this issue 6 years ago • 1 comments

Sort of related to #1

https://github.com/roots/bedrock-autoloader/blob/02e61a6f7ff78c2d77599843617f88156c4c6274/src/Autoloader.php#L12

Should this namespace be Roots\Bedrock\Autoloader ?

austinpray avatar Jun 09 '19 16:06 austinpray

This wouldn't make much difference now I don't think, but I think it's a good idea for use in the future. Now that it's a dedicated package, it might benefit from splitting up a bit into multiple classes or extract a few namespaced utility functions.

Thinking farther down, there could be sub-classes of Autoloader which might implement different strategies for autoloading, etc.

Sounds good to me 👍

aaemnnosttv avatar May 17 '20 20:05 aaemnnosttv