-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
Contador de dias uteis junto ao de dias corridos #67
Conversation
… feriados nacionais) - fixes GabrielRF#19
@MatheusPush Vou testar aqui! Muitíssimo obrigado! |
check_update.py
Outdated
pass | ||
data = evento['data'] + ' ' + evento['hora'] | ||
data = f'{evento["data"]} {evento["hora"]}' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
O Rastreiobot hoje roda no Python 3.4, f-strings não vai funcionar :/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Infelizmente não sabia que estava rodando na 3.4, mas este é um ajuste simples de se fazer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
O Rastreiobot hoje roda no Python 3.4, f-strings não vai funcionar :/
3.5.2
(não sei se isso ajuda ou não..)
@GabrielRF ideia desse contador tem mais de ano né? Lembro de discutir na última hacktoberfest inclusive! Valeu @MatheusPush! |
*.lock | ||
.idea |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adiciona um \n
aqui, pls.
feriados_em_dias_uteis = 0 | ||
for feriado in feriados_entre_datas: | ||
feriados_em_dias_uteis += 1 if feriado.weekday() < 5 else 0 | ||
dias_uteis -= feriados_em_dias_uteis |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Esse método já ta grande demais né? O que você acha de exportar a lógica para uma outra função que recebe dois objetos date
e retorna os dias úteis? Acho que uma função dessa cabe no utils.py
e agora podemos fazer testes :)
fixes #19