-
Notifications
You must be signed in to change notification settings - Fork 13
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
Phpsp test marcosptf #11
Conversation
commiting functions tests
fix pull request from testFest,thanks folks @rogeriopradoj @marcosptf @datibbaw @duodraco @ivanrosolen
fix pull request from testFest,thanks folks @rogeriopradoj @marcosptf @datibbaw @duodraco @ivanrosolen
fix pull request from testFest,thanks folks @rogeriopradoj @marcosptf @datibbaw @duodraco @ivanrosolen
fix pull request from testFest,thanks folks @rogeriopradoj @marcosptf @datibbaw @duodraco @ivanrosolen
Oi, Marcos, vou falar em português mesmo para ficar mais fácil para mim, ok? Seguinte: vamos te ajudar a mandar o pull request direto para o repositório oficial (https://github.com/php/php-src.git), certo? Então se ainda não fez:
Depois você tem que pensar para qual branch do repositório oficial seu pull request tem que ser mandado, a pergunta que você sempre tem que fazer é a seguinte:
De acordo com essa sua resposta, você vai escolher a partir de qual branch você trabalha, de qual vai criar o seu branch, e depois mandar o pull request contra esse branch também. Os nomes dos branches seguem abaixo:
E aí você está pronto para mandar direto para o repositório oficial! Agora uma coisa específica de projetos open source, não estou falando especificamente sobre o php-src: sempre vale a pena mandar 'pull requests separados' para 'coisas separadas'. Nesse caso aqui, seria você em vez de mandar um pull request só com testes de duas funções diferentes, quebrar em dois pull requests. Você teria na sua máquina dois branches seus criados, um para os testes da função X e outro para os testes da função Y. Com isso, você consegue mandar cada branch num pull request diferente, beleza? Cara, falei, muito, mas acho que tem o caminho aí. Se quiser, chama a gente de novo, ok? Até mais |
@rogeriopradoj |
hello everyone, i've removed error_reporting(*) like was said, please, merge to master.
thanks folks!!!
@rogeriopradoj
@marcosptf
@datibbaw
@duodraco
@ivanrosolen