Skip to content

feat: add alphanumeric cnpj#741

Open
niltonpimentel02 wants to merge 2 commits into
brazilian-utils:mainfrom
niltonpimentel02:685
Open

feat: add alphanumeric cnpj#741
niltonpimentel02 wants to merge 2 commits into
brazilian-utils:mainfrom
niltonpimentel02:685

Conversation

@niltonpimentel02

@niltonpimentel02 niltonpimentel02 commented Jun 17, 2026

Copy link
Copy Markdown
Member

Descrição

Baseado no PR #723 com algumas melhorias para atender aos critérios do projeto. Deixo aqui registrado o nosso agradecimento ao autor por sua contribuiçã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 #685

@niltonpimentel02 niltonpimentel02 requested review from a team as code owners June 17, 2026 00:32
@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.09%. Comparing base (e1ce34a) to head (ec373fd).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #741      +/-   ##
==========================================
+ Coverage   99.08%   99.09%   +0.01%     
==========================================
  Files          26       26              
  Lines         765      774       +9     
==========================================
+ Hits          758      767       +9     
  Misses          7        7              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@niltonpimentel02

Copy link
Copy Markdown
Member Author

salve @morais90 alguma chance de tu me ajudar e fazer o review desse código?

Comment thread brutils/__init__.py
# CNPJ
"format_cnpj",
"generate_cnpj",
"generate_alphanumeric_cnpj",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Acho que não faz muito sentido ter outra função, CNPJ vai ser alfanumérico de qualquer forma daqui a pra frente, então generate_cnpj deveria suportar isso. Que acha?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

faz sentido isso que tu falou.. talvez seria interessante deixar a função original do jeito que está e passar um param se quer usar essa função do alphanumeric e chamar essa generate_alphanumeric_cnpj.. não podemos perder de vista que a lib também é usada de forma educacional então pra deixar mais simples e entendível antes de colocar tudo na mesma função.. o que tu acha? se for uma m.. essa minha ideia pode falar aí.. quero fazer o melhor mas de forma mais simplista possível.. inevitavelmente vai gerar um breaking changes na próxima release então acho que isso ficaria ok.. pode jogar umas ideias de código aí se achar necessário pra essa função ou nomes de params e afins.

@morais90

Copy link
Copy Markdown
Contributor

salve @morais90 alguma chance de tu me ajudar e fazer o review desse código?

Certamente senhor

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.

Suporte ao novo CNPJ Alfanumérico

2 participants