Skip to content
This repository has been archived by the owner on Nov 2, 2024. It is now read-only.

Ficha de conteudo de importação #4

Merged
merged 9 commits into from
Nov 4, 2015
Merged

Conversation

mileo
Copy link
Member

@mileo mileo commented Nov 4, 2015

No description provided.

@mileo
Copy link
Member Author

mileo commented Nov 4, 2015

@biancatella Por favor repita o PR para o python3

@mileo
Copy link
Member Author

mileo commented Nov 4, 2015

@landscape-bot
Copy link

Code Health
Repository health decreased by 0.48% when pulling f0d82c8 on kmee:fci into f6728aa on sped-br:python-2.7.

@@ -8,6 +8,7 @@ install:
- pip install coveralls
script:
- coverage run --source=sped setup.py test
- python test/fci_teste.py
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

O path do arquivo aqui está errado.
Deve ser python test/fci_test.py, por isso está quebrando o build do travis.

@sergiogarciadev
Copy link
Member

Em breve efetuo os testes com py3, dai efetuo o merge e gero uma nova versão.

Correção no nome do arquivo de teste.
@biancatella
Copy link
Contributor

Obrigada, @Sergio-Garcia. Alterei o nome do arquivo de teste.

sergiogarciadev added a commit that referenced this pull request Nov 4, 2015
Ficha de conteudo de importação
@sergiogarciadev sergiogarciadev merged commit eb2ff54 into sped-br:python-2.7 Nov 4, 2015
@biancatella
Copy link
Contributor

https://landscape.io/github/kmee/python-sped/1/modules/sped/fci/arquivos.py#L13
https://landscape.io/github/kmee/python-sped/1/modules/sped/fci/arquivos.py#L20
Nesse caso, não é possível mudar o nome da classe "ArquivoDigital" pois ele está sendo usada para criar esse tipo de objeto no sistema. Esse forma de implementação já ocorria no ecf por exemplo

https://landscape.io/github/kmee/python-sped/1/modules/sped/fci/arquivos.py#L65
"buffer" é palavra reservada do Python, mas na classe base também encontramos esse erro.
Posso alterar o nome para str_buffer?

@Sergio-Garcia

@sergiogarciadev
Copy link
Member

Os erros do landscape da pra ignorar. Eu tenho que fazer um monte de
refactor pra zerar ele. Logo mais eu testo com o py3 e público isso.

Enviado do Celular
Em 04/11/2015 13:52, "Bianca Tella" notifications@github.com escreveu:

https://landscape.io/github/kmee/python-sped/1/modules/sped/fci/arquivos.py#L13

https://landscape.io/github/kmee/python-sped/1/modules/sped/fci/arquivos.py#L20
Nesse caso, não é possível mudar o nome da classe "ArquivoDigital" pois
ele está sendo usada para criar esse tipo de objeto no sistema. Esse forma
de implementação já ocorria no ecf por exemplo

https://landscape.io/github/kmee/python-sped/1/modules/sped/fci/arquivos.py#L65
"buffer" é palavra reservada do Python, mas na classe base também
encontramos esse erro.
Posso alterar o nome para str_buffer?


Reply to this email directly or view it on GitHub
#4 (comment).

@mileo
Copy link
Member Author

mileo commented Nov 4, 2015

@biancatella Verificou esses itens: https://landscape.io/diff/233215

@sergiogarciadev
Copy link
Member

@mileo eu integrei o landscape e não fiz todos os refactors para ele funcionar adequadamente. Logo, está dando diversos erros nele.
@biancatella deixe que eu verifico isso.

@biancatella
Copy link
Contributor

@Sergio-Garcia legal que vc fez o merge! :)
Mas ainda estou corrigindo alguns itens. Até o final da tarde mando um pull request com as modificações

@sergiogarciadev sergiogarciadev mentioned this pull request Nov 5, 2015
sergiogarciadev pushed a commit that referenced this pull request Mar 5, 2020
Atualização dos registros da SPED EFD e outras implementações.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants