feat: add alphanumeric cnpj#741
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 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. 🚀 New features to boost your workflow:
|
|
salve @morais90 alguma chance de tu me ajudar e fazer o review desse código? |
| # CNPJ | ||
| "format_cnpj", | ||
| "generate_cnpj", | ||
| "generate_alphanumeric_cnpj", |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
Certamente senhor |
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
Comentários Adicionais (opcional)
Issue Relacionada
Closes #685