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

428 #488

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

428 #488

wants to merge 7 commits into from

Conversation

Erysilva98
Copy link

@Erysilva98 Erysilva98 commented Feb 1, 2025

Descrição

Mudanças Propostas

Checklist de Revisão

  • Eu li o Contributing.md
  • Os testes foram adicionados ou atualizados para refletir as mudanças (se aplicável).
  • Foi adicionada uma entrada no changelog / Meu PR não necessita de uma nova entrada no changelog.
  • 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.
  • Eu documentei as minhas mudanças no código, adicionando docstrings e comentários. Instruções
  • O código segue as diretrizes de estilo e padrões de codificação do projeto.
  • 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 #<numero_da_issue>

@Erysilva98 Erysilva98 requested review from a team as code owners February 1, 2025 18:00
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-487 que está criando release, e adicionando o método de validação do renavam no init no mesmo lugar que voce está importando a validação do rg

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

@Erysilva98
Copy link
Author

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

@tiagornandrade
Copy link
Contributor

Boa @Erysilva98 ja mandei o approve 🚀

@Oleari19
Copy link

Oleari19 commented Feb 5, 2025

@Erysilva98 o meu PR deu o mesmo problema que o seu: "Tem o #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??

@Erysilva98
Copy link
Author

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

@igorbenav
Copy link

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

@Erysilva98
Copy link
Author

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

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.

4 participants