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

428

Open Erysilva98 opened this issue 10 months ago • 6 comments

Descrição

Mudanças Propostas

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.
  • [ ] 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>

Erysilva98 avatar Feb 01 '25 18:02 Erysilva98

@tiagornandrade - Realizei as correções de importações, todos os teste passaram e código segue os padrões da contribuição.

Erysilva98 avatar Feb 02 '25 14:02 Erysilva98

Boa @Erysilva98 ja mandei o approve 🚀

tiagornandrade avatar Feb 02 '25 14:02 tiagornandrade

@Erysilva98 o meu PR deu o mesmo problema que o seu: "Tem o https://github.com/brazilian-utils/brutils-python/pull/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" Vc pode me orientar em como fez a correção??

Oleari19 avatar Feb 05 '25 14:02 Oleari19

@Oleari19, seguir a ordem dos arquivos na pasta brutils, realizei a importação da função abaixo das importações do from brutils.pis em all = [ ] segue o mesmo abaixo da # Pis adicionei o #RG e o nome da minha função.

Erysilva98 avatar Feb 05 '25 15:02 Erysilva98

Opa, @Erysilva98, tem como adicionar as informações do template do PR e um título descritivo por favor? Ajuda muito

igorbenav avatar Feb 07 '25 22:02 igorbenav

@igorbenav Sim claro, irei adicionaar as descrições

Erysilva98 avatar Feb 08 '25 15:02 Erysilva98

Importante notar que existem dois PRs abertos atualmente que propõe uma solução para essa mesma issue:

  • https://github.com/brazilian-utils/brutils-python/pull/498
  • https://github.com/brazilian-utils/brutils-python/pull/488

camilamaia avatar May 13 '25 17:05 camilamaia