Conversation
rougeth
left a comment
rougeth
left a comment
There was a problem hiding this comment.
Obrigado pelo PR @silviasfon. Faltou só adicionar o workadays como dependência do projeto. Deixei outro comentário, mas pode ficar para um próximo PR.
| import json | ||
| import sys | ||
| from datetime import date | ||
| from workadays.workdays import networkdays |
There was a problem hiding this comment.
Faltou atualizar o pyproject.toml com a nova dependência, né?
There was a problem hiding this comment.
Faltou só adicionar o poetry.lock atualizado. Por isso falhou o teste.
Warning: The lock file is not up to date with the latest changes in pyproject.toml. You may be getting outdated dependencies. Run update to update them.
apis/apicorreios.py
Outdated
| elif delta_utils > 1: | ||
| data = data + ' – ' + str(delta_utils) + ' dias úteis)' | ||
| else | ||
| data = data + ')' |
There was a problem hiding this comment.
Você acha que seria muito difícil extrair essa funcionalidade para uma função? Minha impressão é que essa parte do parse() poderia ficar muito mais simples se dividíssemos a lógica em funções menores. Facilitaria até para incluir testes.
- refatoração de código e correção de lógica
| elif delta_dias > 0: | ||
| data = data + ')' | ||
|
|
||
| return data |
There was a problem hiding this comment.
Wow! Boa! Só fica melhor se puder adicionar alguns testes pra essa função! 😁
There was a problem hiding this comment.
estou com um pouco de dificuldade de entender exatamente como fazer, mas vou dar uma olhada
There was a problem hiding this comment.
Esse teste talvez seja um bom exemplo, avisa aqui se não tiver dando certo.
3 participants
Issue #19
Adiciona contador de dias úteis nas informações do pacote, baseado nos feriados do Brasil.