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

Fixed file-name capitalization #45

Closed
wants to merge 1 commit into from
Closed

Fixed file-name capitalization #45

wants to merge 1 commit into from

Conversation

ChipWolf
Copy link

@ChipWolf ChipWolf commented Jan 7, 2014

No description provided.

@ChipWolf ChipWolf mentioned this pull request Jan 7, 2014
@GrahamCampbell
Copy link
Member

NO! This brakes autoloading and is a violation of PSR.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling ec251bb on MichaelBanks:patch-3 into 7429e23 on FrenkyNet:master.

@bencorlett
Copy link
Member

I agree with @GrahamCampbell here.

Having said that, most of the issues on this repo are a little obsessive.

@GrahamCampbell
Copy link
Member

Sorry, I know @Michaelbanks irl so we are just bitchin' with each other...

@frankdejonge
Copy link
Member

I'm closing this due to the PSR compatibility break it will cause, this package will move to @php-loep soon which requires these coding standards.

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.

None yet

5 participants