Feat: Create python-app.yml #3
Conversation
Adicionando GitHub Actions e verificações de formatação com a lib Black, teste com pytest e coverage e estilo.
…ions Feat: Create python-app.yml
Adcionando orientações para instalação das verificações de Qualidade de código.
|
Neste pull request, foram adicionadas as seguintes funcionalidades:
|
.github/workflows/python-app.yml
Outdated
|
|
||
| - name: Test with pytest and coverage | ||
| run: | | ||
| pytest --cov=. --cov-report=term |
There was a problem hiding this comment.
falta adicionar a falha se a cobertura estiver abaixo de 100%
--cov=. ele ira ver a cobertura dos repositórios raiz, o correto é usar apenas o repositório app
.github/workflows/python-app.yml
Outdated
| - name: Verify with flake8 | ||
| run: flake8 . | ||
|
|
||
| - name: Test with pytest |
There was a problem hiding this comment.
Não precisa testar 2 vezes, o "job" de baixo já testa o código
.github/workflows/python-app.yml
Outdated
| # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide | ||
| flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics | ||
|
|
||
| - name: Verify with flake8 |
There was a problem hiding this comment.
o Job de cima já verifica o flake8, pode deletar esse job
.docker/docker-compose.yml
Outdated
| @@ -1,46 +0,0 @@ | |||
| services: | |||
There was a problem hiding this comment.
Porque esse arquivo foi deletado?
There was a problem hiding this comment.
o arquivo docker-compose foi excluído erroneamente.
O arquivo foi retornado para o pasta docker.
JessiAraujo
force-pushed
the
JessiAraujo-feature-github-actions
branch
from
f8d3670
to
3d5a091
Compare
| @@ -43,4 +43,4 @@ services: | |||
|
|
|||
| volumes: | |||
| mysql_database: | |||
| name: mysql_database | |||
.gitignore
Outdated
| @@ -175,3 +175,6 @@ pyrightconfig.json | |||
|
|
|||
| # End of https://www.toptal.com/developers/gitignore/api/python | |||
|
|
|||
| app/some_module.py | |||
There was a problem hiding this comment.
Geralmente o gitigonre não é utilizado para ignorar pastas pessoais, essas não devem ser commitadas
ele deve ser utilizado para pastas gerais que não devem subir
vc adicionou 1 pasta e 1 arquivo que não deveriam estár aqui
você pode subir o pytest.ini para o git, para que seus colegas rodem o mesmo comando que sera rodado na esteira quendo eles forem escrever os testes unitários
a pasta tests/ é onde vamos fazer nossos testes unitários, portanto ela não deve estar no gitignore
então vc precisa excluir essas linhas, pode subir o arquivo pytest.ini e pode subir a pasta tests, desde que esteja vazia
Adicionando GitHub Actions padrão para python
Adcionado verificação de formatação com a lib black e padrão pep8
Adcionado verificação do pytest com 100% de coverage
Adcionado verificação do flake8