-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Rename the loader class #39
Comments
That sounds good. Originally, I thought there was going to end up being more in there, but it looks like we might not need more than that, so this rename makes sense. |
Ok, I'll make a PR as soon I can. Want me to move it also under the Autoloader namespace or it can stay where it is? |
Moving to autoloader namespace sounds good. |
👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I see the Loader Class does not actually load a file, but it just locates non-class files. Hence I propose rename it to FileLocator and move it under the CodeIgniter\Autoloader namespace.
If you're ok with it I can take the burden :)
The text was updated successfully, but these errors were encountered: