Skip to content

Github Angola Users#8

Open
guimar86 wants to merge 5 commits into
codingdojoangola:desafiosfrom
guimar86:desafios
Open

Github Angola Users#8
guimar86 wants to merge 5 commits into
codingdojoangola:desafiosfrom
guimar86:desafios

Conversation

@guimar86

Copy link
Copy Markdown
Contributor

Desafio para aquisição da lista de utilizadores do GitHub em Angola

Entrega do desafio relativo ao calculo da idade de uma pessoa, passando o seu ano de nascimento como parâmetro.
Comment thread tests/testAgeCalculator.php Outdated
</form>


<?php

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Referente ao teste, a classe não deve contar html. O html pode estar por exemplo numa página /app/github/index.php. Porque nessa classe dentro de tests deve ser feito o teste usando PHPUnit

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guimar86 falta ajustes para o teste passar no StyleCI.
Dá uma olhada na analise do StyleCI para entender onde ajustar..

@JoseCage JoseCage left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ainda está em falta a página html e o teste do PHPUnit para podermos avançar com a conclusão do exercício.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants