Skip to content

Conversation

@diogohs
Copy link
Contributor

@diogohs diogohs commented Jun 5, 2025

Olá!

Dada a necessidade de uma lib compatível com o novo formato de CNPJ, abri esse PR adicionando o suporte ao CNPJ alfanumérico. Eu tomei cuidado para que todas as modificações propostas não impactem usos e usuários atuais.

  • Altera função na BaseDoc (_validate_input) para validar sequência alfanumérica
  • Adiciona função na BaseDoc (_only_digits_and_letters) para limpar sequências alfanuméricas
  • Altera as funções de cálculo dos DVs, de acordo com o manual da Receita Federal
  • Altera a função geradora de CNPJs permitindo opção com e sem caracteres alfanuméricos (default: somente dígitos)
  • Adiciona alguns testes para alfanumérico
  • Typo no CONTRIBUTING.md

#71

@alvarofpp
Copy link
Owner

Olá, @diogohs . Primeiramente obrigado pela sua contribuição.

O lint apontou alguns erros que precisam ser corrigidos para se adequarem ao repositório. Você pode executar o comando make lint localmente para replicar a mesma operação feita pelo job lint.

De resto, eu testei aqui e aparenta esta tudo okay, parabéns!

@diogohs diogohs force-pushed the feat/cnpj-alfanumerico branch from 984fe30 to 8b12b6e Compare June 6, 2025 01:38
@diogohs
Copy link
Contributor Author

diogohs commented Jun 6, 2025

Olá @alvarofpp, corrigi os erros de commitlint e dois de formatação. Espero que agora esteja correto. Abraço

@alvarofpp alvarofpp merged commit dd3393c into alvarofpp:main Jun 6, 2025
3 checks passed
@diogohs
Copy link
Contributor Author

diogohs commented Jun 6, 2025

Olá @alvarofpp, seria possível você liberar uma nova release no pypi para que possamos já utilizar a lib atualizada?
Obrigado!

@alvarofpp
Copy link
Owner

@diogohs Lancei a versão 1.11.0 do validate-docbr com a sua contribuição.
https://pypi.org/project/validate-docbr/1.11.0/

@diogohs
Copy link
Contributor Author

diogohs commented Jun 7, 2025

Obrigado @alvarofpp

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.

2 participants