lipe14-ops / brasilapy

Brasil API client
MIT License
94 stars 22 forks source link

Adicionar a flexibilidade de trabalhar com CNPJ do tipo filial #11

Closed joepreludian closed 1 year ago

joepreludian commented 1 year ago

Eu descobri um problema ao usar nossa API. Fiz uma pequena correção nos modelos que visa manipular dados que usam CNPJ.

Por favor, @lipe14-ops, verifique se está ok e aprove a PR de acordo. =)

Abraço!

codecov-commenter commented 1 year ago

Codecov Report

Merging #11 (397f02a) into master (6f2efd1) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #11   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines          264       264           
=========================================
  Hits           264       264           
Impacted Files Coverage Δ
brasilapy/models/cnpj.py 100.00% <100.00%> (ø)
joepreludian commented 1 year ago

@lipe14-ops eu fiz um ajuste no precommit... resolvi uma dependencia que estava quebrando o nosso CI/CD. Voce poderia analisar e fazer o merge assim que possível? (ou me add como mantenedor pra que eu possa fazê-lo mais rapidamente?)

[pq to usando essa lib no meu projeto pessoal e acredito que eu possa achar mais problemas, dai ja vou corrigindo]

joepreludian commented 1 year ago

Ah,, @lipe14-ops.... Eu vi que o sonarQube não está configurado. Depois, quando tiver uma chance, verifica isso pra ficarmos com o sonarcloud tbm. Abraço. =D

lipe14-ops commented 1 year ago

O sonarcloud está com o problema do token... Mas a variável estpa declarada la no secrets > actions com o nome e o valor... MAS NÃO PASSA NOS TESTES

o que pode ser?

joepreludian commented 1 year ago

Podemos tentar descobrir. será que não foi desativado algo no sonar? Na verdade os testes estão passando, mas o quality gate n.

Faça assim... aprove esse pr e faça o merge... que eu vou tentar ver direitinho esse problema... Mas vendo os logs, achoq que é relacionado ao token.

joepreludian commented 1 year ago

@lipe14-ops Set the SONAR_TOKEN env variable.' Os dois possiveis pontos sao o SONAR_TOKEN não estar configurado no CI, no secrets do seu repo, ou no Sonarcloud (se foi revogado, ou deu algum problema)

lipe14-ops commented 1 year ago

ÓTIMO!!!