Skip to content
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

Closed
wants to merge 3 commits into from

Conversation

MatheusPush
Copy link

  • Removendo apenas com feriados nacionais e finais de semana
  • Ajustes de pep8

fixes #19

@GabrielRF
Copy link
Owner

@MatheusPush Vou testar aqui! Muitíssimo obrigado!
Vai ser muito bacana ter esse contador!!

check_update.py Outdated
pass
data = evento['data'] + ' ' + evento['hora']
data = f'{evento["data"]} {evento["hora"]}'
Copy link
Collaborator

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 :/

Copy link
Author

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

Copy link
Owner

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..)

check_update.py Outdated Show resolved Hide resolved
check_update.py Outdated Show resolved Hide resolved
@rougeth
Copy link
Collaborator

rougeth commented Oct 17, 2019

@GabrielRF ideia desse contador tem mais de ano né? Lembro de discutir na última hacktoberfest inclusive! Valeu @MatheusPush!

*.lock
.idea
Copy link
Collaborator

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
Copy link
Collaborator

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 :)

@Bigua Bigua closed this Aug 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Contador de dias úteis
4 participants