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

Release v<2.3.0> #487

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Release v<2.3.0> #487

wants to merge 3 commits into from

Conversation

Oleari19
Copy link

@Oleari19 Oleari19 commented Feb 1, 2025

Descrição

Utilitário is_valid_renavam #430

Implementação da issue #430

Mudanças Propostas

Criação da função de validar o renavam e seus rescpectivos testes, alterações também na documentação conforme pede o arquivo de contribuição

Checklist de Revisão

  • [ X] Eu li o Contributing.md
  • [ X] Os testes foram adicionados ou atualizados para refletir as mudanças (se aplicável).
  • [ X] Foi adicionada uma entrada no changelog / Meu PR não necessita de uma nova entrada no changelog.
  • [ X] A documentação em português foi atualizada ou criada, se necessário.
  • [ X] Se feita a documentação, a atualização do arquivo em inglês.
  • [ X] Eu documentei as minhas mudanças no código, adicionando docstrings e comentários. Instruções
  • [ X] O código segue as diretrizes de estilo e padrões de codificação do projeto.
  • [ X] Todos os testes passam. Instruções
  • [ X] O Pull Request foi testado localmente. Instruções
  • [ X] Não há conflitos de mesclagem.

Comentários Adicionais (opcional)

Ao rodar 'make test', foi preciso executar os comandos 'pip install holidays' e 'pip install num2words', seria important documentar.

Além disso, não consegui rodar apenas o meu arquivo de testes criados, devido a isso fiz usando pytest e ao final rodei o 'make test' e passou todos.

Issue Relacionada

Closes #430

Oleari19 and others added 2 commits February 1, 2025 07:38
…ation of RENAVAM and its respective tests. Additionally, the appropriate documentation as required by the contribution file.
@Oleari19 Oleari19 requested review from a team as code owners February 1, 2025 14:12
@tiagornandrade tiagornandrade mentioned this pull request Feb 2, 2025
10 tasks
Copy link
Contributor

@tiagornandrade tiagornandrade left a comment

Choose a reason for hiding this comment

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

Tem o PR - 488 que está adicionando o método de validação do rg no init no mesmo lugar que voce está importando a validação do renavam

@Oleari19
Copy link
Author

Oleari19 commented Feb 5, 2025

@tiagornandrade não entendi muito o problema... peguei todas as alterações do #488 e adicionei na branch 430. Rodei todos os testes e passaram!! No aguardo de um feedback

@Oleari19
Copy link
Author

Oleari19 commented Feb 5, 2025

Sugestão de Issue: fazer as devidas alterações para o 'test_date_utils.TestIsHoliday' funcionar. Segue o resultado final que aparece no terminal ao realizar o comando 'make test', todos passam menos ele

======================================================================
FAIL: test_data_passada_futura (test_date_utils.TestIsHoliday)

Traceback (most recent call last):
File "/home/maria/Documentos/teste/Projeto/brutils-python/tests/test_date_utils.py", line 63, in test_data_passada_futura
self.assertTrue(is_holiday(datetime(2150, 12, 25))) # Ano futuro
AssertionError: False is not true


Ran 137 tests in 2.391s

FAILED (failures=1)

@tiagornandrade
Copy link
Contributor

@Oleari19 não é um problema na resolução da issue, foi só um lembrete que tinha o seu PR e outro fazendo ajuste no mesmo arquivo. Mas o PR anterior já tratou o possível conflito

Copy link
Contributor

@tiagornandrade tiagornandrade left a comment

Choose a reason for hiding this comment

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

LGTM

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.

Validação do RENAVAM
2 participants