Skip to content

Conversation

carusogabriel
Copy link
Contributor

Hi there :octocat:

With this PR, I've cleaned up the whole whitespaces in php-src C files 🚮

If this gets accepted, would be my pleasure to run with other types of file 😄

(I just don't know if I committed in the right branch 😅)

Signed-off-by: Gabriel Caruso <carusogabriel34@gmail.com>
Signed-off-by: Gabriel Caruso <carusogabriel34@gmail.com>
Signed-off-by: Gabriel Caruso <carusogabriel34@gmail.com>
Signed-off-by: Gabriel Caruso <carusogabriel34@gmail.com>
Signed-off-by: Gabriel Caruso <carusogabriel34@gmail.com>
@carusogabriel
Copy link
Contributor Author

carusogabriel commented Dec 31, 2017

Big thanks to @javiereguiluz’s tip that I'll help review: passing ?w=1 to Files Changed tab, we can ensure that all files only contains Whitespace-only changes. 🙏

Signed-off-by: Gabriel Caruso <carusogabriel34@gmail.com>
@php-pulls
Copy link

Comment on behalf of cmb at php.net:

Thanks, Gabriel. I've applied the PR in 6400264. And yes, all non-bug-fixes should target master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants