Adaptacoes para a versao limpada da nfelib https://github.com/akretion/nfelib #30
Conversation
rvalyi
force-pushed
the
nfe-cleanup2
branch
2 times, most recently
from
16f8ce4
to
5229a59
Compare
|
Pessoal, tem um erro no Travis com o isort que ja tem na branch master, mas o restante ficou verde no meu PR. Mas de inicio, acho que a prioridade é ter um nfelib limpo gerido de forma determinista e fácil sem gambiarra depois de gerido. |
|
@gabrielcardoso21 @luismalta @mileo podem revisar por favor ? |
|
uṕ |
|
ping @mileo @gabrielcardoso21 @luismalta na verdade está PR tem mais prioridade que a PR OCA/l10n-brazil#981 |
|
ping @mileo @gabrielcardoso21 |
|
ping @rvalyi |
|
@rvalyi favor revisar os comentários do akretion/nfelib#23 |
|
Outra coisa pode dar uma olhada nos testes? |
|
@mileo esta PR está na mesma situação da erpbrasil/erpbrasil.edoc.pdf#6 ? Se sim, poderiamos seguir da mesma forma como foi feito com a erpbrasil.edoc.pdf. cc @rvalyi |
Estou vendo se chegamos a um consenso no akretion/nfelib#23 para deixarmos tudo no mesmo formato. |
@mileo Eu concordo com a definição de um formato padrão mas como comentei, se pudermos seguir da mesma forma como foi feito com a erpbrasil.edoc.pdf (merge e os tapas do @mileo para deixar passando nos testes e fazendo bump da versão) acho que vale a pena para não ficar parada esta PR. O que acha? talvez eu não esteja observando algum detalhe mas não me parece que a padronização da nfelib seria um entrave para seguir com esta PR. |
O formato padrão garante que vamos poder automatizar coisas, quem sabe botar um BOT em tudo para fazer essa rotina de fechamento de versão e publicar no pypi. Ou pelo menos todos sabendo como fazer manualmente já facilita a vida. |
Sim. Concordo. No caso desta PR como teria que tratar o erro que deu ? |
Só rodar esse comando: isort --verbose --recursive src tests setup.py Ele corrige automaticamente. |
@rvalyi independente de como vai ficar, (tox, pytest e afins) consegue rodar o comando sugerido pelo @mileo para ver se passa e com isso poder liberar para merge ? |
|
@mileo pelo jeito não rolou não. |
|
@rvalyi aee.. bora fazer o merge ? :D |
|
ufa foi!! |
|
@mileo pode fazer o merge por favor? |
|
up |
|
Bom dia pessoal. Algum impedimento para o merge desta PR ? |
|
@mileo pode fazer o merge ? |
|
Pessoal.. algum impedimento para seguir com o merge ? |
gabrielcardoso21
force-pushed
the
master
branch
from
db8e0e6
to
b8eeb12
Compare
Work In Progress!
ainda tem um bug em tests/test_erpbrasil_edoc.py no test_ultimo_nsu(), ainda nao sei porque.