Mudança no método clean_none_params para ignorar somente valores None

Japle / python-pagseguro

Integração com a API v2 de pagamentos e notificações do Pagseguro utilizando requests.

MIT License

172 stars 69 forks source link

Mudança no método clean_none_params para ignorar somente valores None #52

Closed jdcarvalho closed 7 years ago

jdcarvalho commented 7 years ago

Conforme disposto na issue #51 segue o pull request

ellisonleao commented 7 years ago

opa @jdcarvalho teve 2 testes que falharam, será que daria pra corrigir ?

jdcarvalho commented 7 years ago

Verdade @ellisonleao vou corrigí-los

coveralls commented 7 years ago

Coverage decreased (-0.3%) to 60.22% when pulling fcf8dd30550a67304df0439f50cf96376aedd2fc on jdcarvalho:master into 7dd7189cf02b3194412d9d2d6d3c7ea818cdede3 on rochacbruno:master.

jdcarvalho commented 7 years ago

@ellisonleao fiz um commit aqui para que o método aceite os valores booleanos (mesmo sendo False) além dos valores válidos (not None). Rodei os testes e eles passaram.

Desculpe pelo envio com erros anteriormente. Não havia instalado os requisitos de desenvolvimento e rodado os testes no primeiro commit.

Não sei se o hint do coveralls é impeditivo para o merge, qualquer coisa só falar,

Atenciosamente,

João Carvalho

ellisonleao commented 7 years ago

O coveralls a gente resolve depois :)

Valeu @jdcarvalho ! 🍻