Closed paulohrpinheiro closed 7 years ago
Ei @paulohrpinheiro , valeu pela contribuição, cara. :) Mas você pode fazer um update do seu PR? Eu fiz uma pequena mudança no travis para executar os testes corretamente. Abs.
Olá @matheuscas. Tudo em paz? O que devo atualizar? Abraço!
@paulohrpinheiro Acho que basta você fazer um pull do master e subir as alterações no seu branch mesmo que vem para o PR. Aí vai rodar o Travis de novo e acho que vai passar numa boa. Daí eu olho com mais calma o PR. Blz?
Olá, Matheus. Para as minhas necessidades este patch está OK, mas como você também usa Python2, ele não é adequado. Vou pensar em algo que atenda às duas versões. Se você tiver uma ideia, por favor me avisa.
@paulohrpinheiro No master, ele já está passando nos testes para o python 3.6, então a lib já é compatível com python 3. Está dando algum erro aí para você quando você atualiza do master?
Então, no Python3 está tudo OK (que por acaso é o que uso :) ) mas para o Python2 não funcionou.
Ah, vi aqui agora no Travis. Seu problema é o maketrans
. Aí eu acho q se você quiser manter o maketrans
, você vai ter que verificar na hora se é Python 2 ou 3. Se for o 2, você tem que importar a função de string desse jeito: from string import maketrans
. Se se for o 3 vc usa como está. Encapsula o maketrans
em uma função e nela faz a verificação das versões. O que acha?
Pra não sujar o que já existe, acredito que seja melhor, num arquivo compatible.py
por exemplo, criar a função que faz a limpeza, com a verificação de versão (acho que é melhor usar o six
, apesar de ser mais uma dependência a instalar). Assim o código atual permanece limpo, e o que trata de versões fica isolado. Que acha?
Merging #10 into master will increase coverage by
2.69%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #10 +/- ##
==========================================
+ Coverage 75.38% 78.08% +2.69%
==========================================
Files 6 7 +1
Lines 65 73 +8
==========================================
+ Hits 49 57 +8
Misses 16 16
Impacted Files | Coverage Δ | |
---|---|---|
pycpfcnpj/cpfcnpj.py | 100% <100%> (ø) |
:arrow_up: |
pycpfcnpj/compatible.py | 100% <100%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 2e538d1...7812253. Read the comment docs.
Agora consegui arrumar a bagunça que fiz :)
Esse PR permite à biblioteca analisar números de documentos que contenham sinais de pontuação como
.
, '-', ou '/'.