Skip to content
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

Composer.json - use psr-4 autoload #10

Closed
wants to merge 3 commits into from
Closed

Composer.json - use psr-4 autoload #10

wants to merge 3 commits into from

Conversation

ovr
Copy link

@ovr ovr commented May 25, 2016

No description provided.

@clue
Copy link
Owner

clue commented May 25, 2016

Thanks for your PR! Just trying to understand this, what is the motivation for this change?

@ovr
Copy link
Author

ovr commented May 26, 2016

To understand changes It's need to review vendor/composer directory with autoload

PSR-0 seems to be legacy standart

  • Support fake namespaces with _ for PHP < 5.3

https://www.sitepoint.com/battle-autoloaders-psr-0-vs-psr-4/

@ovr
Copy link
Author

ovr commented May 26, 2016

I will review why build fails...

@ovr
Copy link
Author

ovr commented May 27, 2016

refs #8
ping @clue

@clue
Copy link
Owner

clue commented Aug 3, 2016

Changes LGTM 👍

The last commit certainly makes sense, but I'd rather see this in a separate PR, because it is currently unrelated. Can you squash your commits? 👍

@clue clue deleted the branch clue:master July 26, 2024 14:58
@clue clue closed this Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants