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

Autoload with PSR-4 #43

Merged
merged 1 commit into from
Aug 23, 2015
Merged

Autoload with PSR-4 #43

merged 1 commit into from
Aug 23, 2015

Conversation

vinkla
Copy link
Owner

@vinkla vinkla commented Aug 11, 2015

Since the PSR-0 is deprecated we should use to PSR-4 instead.

Deprecated - As of 2014-10-21 PSR-0 has been marked as deprecated. PSR-4 is now recommended as an alternative - http://www.php-fig.org/psr/psr-0/

@GrahamCampbell
Copy link
Contributor

👎 for moving any files. It's not needed.

@vinkla
Copy link
Owner Author

vinkla commented Aug 11, 2015

@GrahamCampbell I've updated the PR to not moving any files.

/* phpunit tests/HashidsTest.php */
namespace Hashids\Tests;

use Hashids\Hashids;

class HashidsTest extends \PHPUnit_Framework_TestCase {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would be better to import this imo instead of having a leading slash

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, I'll fix that.

Since the PSR-0 is deprecated we should use to PSR-4 instead.
@jd327
Copy link
Collaborator

jd327 commented Aug 17, 2015

Guys, I haven't forgotten about this. Bear with me, as I'm catching up with my backlog.

@vinkla
Copy link
Owner Author

vinkla commented Aug 17, 2015

No worries :) This PR will add PSR-4 and drop PSR-0 without breaking anything.

jd327 pushed a commit that referenced this pull request Aug 23, 2015
@jd327 jd327 merged commit 50a3f2e into vinkla:master Aug 23, 2015
@jd327
Copy link
Collaborator

jd327 commented Aug 23, 2015

Thanks for cleaning up the tests.

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.

3 participants