Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

php.ini text fixes #233

Merged
merged 2 commits into from Nov 25, 2012

Conversation

Projects
None yet
4 participants

BryanH commented Nov 12, 2012

(Note: no code changes, just text changes) I corrected some spelling/grammar errors and rewrote some sentences to make them less cumbersome and easier to read.

For a sanity check, I tested the merge against the latest release branch (v 5.4.9) to make sure it wouldn't have any conflicts. As expected, it didn't.

Bryan Hanks, PMP added some commits Nov 12, 2012

Bryan Hanks, PMP Corrected grammar/spelling of text in php.ini files
There are no actual code changes here; I corrected grammar and spelling
errors in the php.ini-production and php.ini-development files. I also
rewrote some sentences to make them less cumbersome.
d0727b5
Bryan Hanks, PMP Corrected spelling of 'shortcut' e7f8f7c

powtac commented Nov 21, 2012

Cool! I love this kind of "bugfixes", everybody can add something valuable to PHP!

@php-pulls php-pulls merged commit e7f8f7c into php:master Nov 25, 2012

1 check failed

default The Travis build failed
Details
Owner

laruence commented Nov 25, 2012

applied, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment