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

generate voter id #220

Open patricia-salles opened this issue 1 year ago • 2 comments

Descrição

Objetivo dessa PR é incluir a função que será responsável por gerar um título de eleitor válido e aleatório.

Mudanças Propostas

Inclusão da função generate_voter_id() e ajuste na função _calculate_vd2() para incluir a verificação se o resto da divisão for igual a 10.

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)

Issue Relacionada

Closes #<numero_da_issue>

patricia-salles avatar Feb 09 '24 00:02 patricia-salles

Codecov Report

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

Project coverage is 100.00%. Comparing base (0f12511) to head (7f252c9).

:exclamation: Current head 7f252c9 differs from pull request most recent head 202092c

Please upload reports for the commit 202092c to get more accurate results.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #335   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines          313       327   +14     
=========================================
+ Hits           313       327   +14     

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

codecov[bot] avatar Feb 09 '24 00:02 codecov[bot]

Olá @antoniamaia, adicionei a nova PR da issue para gerar um número de título de eleitor :)

patricia-salles avatar Feb 09 '24 00:02 patricia-salles

Oii @patricia-salles, mandou bem demais vendo que tinha faltado um edge case no cálculo do dv2, valeu!

A gente fez um pair aqui e arrumou os detalhes que faltavam, agora tá prontinho para dale!

Obrigada pela persistência, essa issue rendeu haha

Que massa que consegui contribuir, fico feliz! Se tiver projetos futuros precisando de contribuição, posso tenta contribuir no meu tempo livre. Valeu :)

patricia-salles avatar Jul 01 '24 23:07 patricia-salles