search

raelgc / template

PHP Template
http://raelcunha.com/template.php
GNU Lesser General Public License v2.1
73 stars 42 forks source link

Simple object test #43

Closed masakik closed 4 years ago

raelgc commented 4 years ago

@masakik Se vc puder, manda essas mudanças no próprio branch 42-tests (ainda não fiz merge dele). Vamos tentar centralizar por lá, tudo num mesmo branch.

masakik commented 4 years ago

mandei no branch. Aproveitando strict_types está disponível a partir do php 7.0.0 então não sei se causa algum problema de compatibilidade com versões anteriores.

masakik commented 4 years ago

se preferir podemos ir conversando no telegram ou whats.

raelgc commented 4 years ago

@masakik Ótima observação: vou remover os strict types, já que no composer estamos falando que a classe Template usa php 5.3. Talvez eu também deva mudar a versão do PHPUnit pra usar uma compatível com PHP 5.3.

Quanto as suas mudanças: acho que vc esqueceu de fazer o push, pois não vejo essas modificações no branch 42-tests, apenas no pull request.

masakik commented 4 years ago

No início desse PR: masakik wants to merge 1 commit into raelgc:42-tests from masakik:42-test-simpleObject. Desculpe mas branches remotos tenho pouca prática então achei que era isso. Se não for não sei onde mando para essa branch específica.

raelgc commented 4 years ago

Ah, você abriu um PR de um branch pra outro, entendi. Deixa eu ver se consigo colocar vc como parte do time, aí vc consegue puxar o repo original e trabalhar direto no branch 42.