search

radar-parlamentar / radar

Estamos de mudança para o GitLab: https://gitlab.com/radar-parlamentar/radar.
http://radarparlamentar.polignu.org
84 stars 44 forks source link

Migração do módulo exportadores para python3 (Com rebase) #428

Closed PedroHenriqueSales closed 7 years ago

diraol commented 7 years ago

Pessoal do time @radar-parlamentar/unb-mes-2017-1 , os testes ainda estavam quebrando (vejam em: https://travis-ci.org/radar-parlamentar/radar/builds/231319447?utm_source=github_status&utm_medium=notification)

O problema era uma dependência (distribute) que estava "pinada" no requirements e que parece não ser compatível com python3. Eu verifiquei e não estamos utilizando-a em nenhum lugar no nosso projeto, então ela provavelmente era uma dependência "de terceiros". Removi ela e os testes parecem ter passado (localmente pelo menos). Ai já que havia feito isso, já fiz o merge desse PR de vocês. =)

Talvez vocês precisem fazer um rebase nas demais branchs de vocês aonde estão trabalhando.

Valeu!

suzukijessica commented 7 years ago

@diraol, você fez o merge com a master, está correto isso? Continuaremos mandando os PR para a branch migrate_to_python3.

Valeu

diraol commented 7 years ago

Ops! @suzukijessica obrigado pelo toque! Falha minha....

PedroHenriqueSales commented 7 years ago

Ops... na verdade foi falha minha :p Posso fazer o PR para a branch correta agora?

diraol commented 7 years ago

@PedroHenriqueSales já que eu já fiz o merge na master, não sei se vale "desfazer". Então fiz o rebase da migrate_to_python3, passando o merge para ela tbm.

diraol commented 7 years ago

@leonardofl Isso pode dar problema na versão em produção (pq jogamos algo na master)?

leonardofl commented 7 years ago

A princípio tudo q é comittado no master dispara o deploy. Então potencialmente pode dar problema. Só q normalmente nosso Jenkins tá fora do ar, oq é o caso agora, então não deve ter dado problema.