Skip to content

Adjustment for alphanumeric CNPJ validation. Resolves #29#30

Merged
paemuri merged 1 commit into
paemuri:mainfrom
Ropelatto:validate-alphanumeric-cnpj
Jun 21, 2025
Merged

Adjustment for alphanumeric CNPJ validation. Resolves #29#30
paemuri merged 1 commit into
paemuri:mainfrom
Ropelatto:validate-alphanumeric-cnpj

Conversation

@Ropelatto

Copy link
Copy Markdown

Alteração para a validação de CNPJ no novo padrão alfanumérico.

Resolves #29

@paemuri

paemuri commented Jun 21, 2025

Copy link
Copy Markdown
Owner

Agradeço pela contribuição! No caso, acabei optando pelo PR #31 por ser mais simples e usar mais do código-fonte original.

@paemuri

paemuri commented Jun 21, 2025

Copy link
Copy Markdown
Owner

Na verdade, ao observar teu código, notei que o código que foi para main possui um problema: estamos utilizando unicode.IsLetter erroneamente, enquanto você fez uma comparação de runas mais simples. Tomarei a liberdade de usar do seu PR como correção, mantendo sua contribuição.

@paemuri paemuri force-pushed the validate-alphanumeric-cnpj branch from 3b0ed42 to 37d2bda Compare June 21, 2025 12:53
@paemuri paemuri merged commit fa4eb35 into paemuri:main Jun 21, 2025
38 checks passed
@paemuri

paemuri commented Jun 21, 2025

Copy link
Copy Markdown
Owner

Parece que, mesmo você sendo o autor do commit (imagem), não ficou como contribuinte do projeto. :/

image

[edit] Talvez a tua conta do Github não esteja atrelada a esse endereço de email?

@Ropelatto

Ropelatto commented Jun 23, 2025

Copy link
Copy Markdown
Author

Olá. Tudo bem por ter optado pelo outro PR. Não analisei a fundo o commit do outro PR, mas olhando superficialmente, me parece que foi efetuado apenas ajustes no regex (me perdoe se estiver errado). No meu PR, eu fiz ajuste no cálculo do dígito verificador do CNPJ validando tanto do padrão vigente, quanto do novo padrão. De qualquer forma, foi um prazer colaborar.

Sobre não aparecer como contribuinte, realmente o git no meu computador está configurado com uma conta diferente do meu Github. Não me atentei a isso.

@paemuri

paemuri commented Jun 23, 2025

Copy link
Copy Markdown
Owner

No caso, o código em main parece atender ambos os padrões já; o toInt existente retorna os valores corretos já (ao que tudo indica, pensaram na tabela ASCII ao decidirem os valores). Inclusive, adicionei um exemplo de um dos documentos oficiais como caso de teste e funcionou normalmente.

Sobre a contrib, se você ir nas configurações da sua conta do Github e incluir o email ronaldo@mrsistemas.com.br lá, provavelmente tudo se ajustará. 👀

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.

Atualização na validação do CNPJ alfanumérico

2 participants