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

Typofixes #332

Closed
wants to merge 16 commits into from
Closed

Typofixes #332

wants to merge 16 commits into from

Conversation

vlajos
Copy link
Contributor

@vlajos vlajos commented Apr 23, 2013

Mostly in non "living" code. Anyway a review is needed...

@dsp
Copy link
Member

dsp commented Apr 26, 2013

This is great, thanks for taking the time to fix those little things. Any chance to rebase the pull request so it is opened against PHP-5.5 branch? I would love to merge this into 5.5. Thanks

@ghost ghost assigned dsp Apr 26, 2013
@vlajos
Copy link
Contributor Author

vlajos commented Apr 26, 2013

Ehh. I am not a big git expert... I tried... I created a new branch typofixes-PHP-5.5 i switched to my orig typofixes branch I typed the git rebase typofixes-55. It seemed to be working, but push isnt copying the changes back to github. I dont understand... Can you help me?

vlajos@asusl:~/github/php-src$ git branch
  master
  typofixes
* typofixes-PHP-5.5
vlajos@asusl:~/github/php-src$ git log --all|head -30
commit 70fd113af70a6bfc7bcd8e3085170cdab25c7799
Author: Veres Lajos 
Date:   Tue Apr 23 11:26:46 2013 +0100
    typofixes (occurrence 2)
commit fde24e1cfd94cc71fc60d6be58318f6818fbe5b0
Author: Veres Lajos 
Date:   Tue Apr 23 11:20:05 2013 +0100
    typofixes (accommodate)
...
vlajos@asusl:~/github/php-src$ git push origin typofixes-PHP-5.5
Everything up-to-date
vlajos@asusl:~/github/php-src$ 

@vlajos vlajos mentioned this pull request May 8, 2013
@vlajos
Copy link
Contributor Author

vlajos commented May 8, 2013

Recreated on a whole new branch: #341

I think it is okey... but not so sure.

@php-pulls
Copy link

Comment on behalf of stas at php.net:

duplicate

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