Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Nov 14, 2019. It is now read-only.

Switch autoloader from PSR-0 to PSR-4 #32

Closed
wants to merge 2 commits into from
Closed

Conversation

jmgq
Copy link
Contributor

@jmgq jmgq commented Oct 23, 2014

Autoloading was changed to PSR-4, as requested in #26.

@fabpot
Copy link
Member

fabpot commented Nov 5, 2014

I'm against changing the directory structure. It makes things not clear enough about the namespaces. You can switch to PSR-4 if you want but let's keep the directory structure intact.

@javiereguiluz
Copy link
Member

@jmgq we want to finish the Symfony installer this week, so I wonder if you have time to do the changes proposed by Fabien. The idea is to switch to PSR-4 but maintaining the same directory structure. If you need some help, you can check out this PR from the Symfony repository which does exactly that.

@jmgq
Copy link
Contributor Author

jmgq commented Nov 18, 2014

Thanks for the link, @javiereguiluz. I've changed the directory structure to the original one. I've also merged this commit locally into the latest version of master and tested that it works correctly.

@fabpot
Copy link
Member

fabpot commented Nov 19, 2014

Thank you @jmgq.

@fabpot fabpot closed this in 45f6f86 Nov 19, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants