Skip to content

Conversation

@Ropelatto
Copy link

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

Resolves #29

@paemuri
Copy link
Owner

paemuri commented Jun 21, 2025

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
Copy link
Owner

paemuri commented Jun 21, 2025

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
Copy link
Owner

paemuri commented Jun 21, 2025

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
Copy link
Author

Ropelatto commented Jun 23, 2025

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
Copy link
Owner

paemuri commented Jun 23, 2025

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