brutils-python icon indicating copy to clipboard operation
brutils-python copied to clipboard

Adiciona validação do RENAVAM - Issue 430

Open LucasAlvws opened this issue 1 year ago • 4 comments

Descrição

Mudanças Propostas

Checklist de Revisão

  • [x] Eu li o Contributing.md
  • [ ] 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
  • [ ] O Pull Request foi testado localmente. Instruções
  • [ ] Não há conflitos de mesclagem.

Comentários Adicionais (opcional)

Issue Relacionada

Closes #430

LucasAlvws avatar Oct 23 '24 16:10 LucasAlvws

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.79%. Comparing base (4af1cdc) to head (347fb94). Report is 18 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #440   +/-   ##
=======================================
  Coverage   99.78%   99.79%           
=======================================
  Files          18       19    +1     
  Lines         472      487   +15     
=======================================
+ Hits          471      486   +15     
  Misses          1        1           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Oct 23 '24 16:10 codecov[bot]

Oi @LucasAlvws ! Uma dica, coloque um título descritivo na sua PR pra aumentar as chances de revisão (tipo "Adiciona validação do RENAVAM")

Aí, na descrição da PR vc pode usar um dos atalhos do github tipo Closes #430 pra ligar sua PR à issue original, e fechar a issue automaticamente quando sua PR for mesclada.

Só não vou revisar o código pq não sou muito familiar com essa parte 😄

melissawm avatar Oct 24 '24 15:10 melissawm

Oi @LucasAlvws ! Uma dica, coloque um título descritivo na sua PR pra aumentar as chances de revisão (tipo "Adiciona validação do RENAVAM")

Aí, na descrição da PR vc pode usar um dos atalhos do github tipo Closes #430 pra ligar sua PR à issue original, e fechar a issue automaticamente quando sua PR for mesclada.

Só não vou revisar o código pq não sou muito familiar com essa parte 😄

A certo, obrigado pela dica, vou fazer isso sim!!

LucasAlvws avatar Oct 26 '24 14:10 LucasAlvws

[PT-BR] Este PR está inativo há muito tempo. Caso não seja atualizado, será fechado em 7 dias. [EN] This PR has been inactive for a long time. If no updates are made, it will be closed in 7 days.

github-actions[bot] avatar Mar 10 '25 00:03 github-actions[bot]

Oi, @LucasAlvws ! Estou fechando o PR por inatividade, mas fique à vontade para reabrir caso queira retomar o trabalho. 😊

camilamaia avatar May 13 '25 17:05 camilamaia