Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Correção e exemplo de geração de XML #2

Open
wants to merge 14 commits into
base: generated
Choose a base branch
from

Conversation

mileo
Copy link

@mileo mileo commented Mar 21, 2018

No description provided.

@rvalyi
Copy link
Member

rvalyi commented Mar 21, 2018

Valeu pela contribuiçao @mileo !

bom, faltou eu explicar isso ainda: a ideia eh que vc faz um git checkout script e roda o ´./generate´ a partir de la. ele gera a branch generated sozinho e ainda faz o merge da branch packaging que contem o README e as outras coisas nao gerado. Ou seja, a branch 'generated' nao deve receber merges ou commits mas deve ter push -f. Depois que a lib for madura, poderiamos conservar as branches generated de determinadas versoes sim para ajudar a rastrear a evoluiçao. Vou accrescentar essa logica no README. Vc pode refazer o PR nas branches script e packaging. Mas se vc nao tiver tempo, posso faze-lo sem problema.

@mileo
Copy link
Author

mileo commented Mar 21, 2018 via email

@rvalyi
Copy link
Member

rvalyi commented Mar 21, 2018

outra coisa, se o XSD precisar de um patch, eh interessante que esse seja aplicado pelo script generate na branch script mesmo, assim tudo fica reproduzivel, desde o download do XSD. Incrivel esses palhaços da fazenda nem checar os XSD's deles...

@rvalyi
Copy link
Member

rvalyi commented Jun 8, 2018

@mileo vi que vc fez mais commits... Porem realmente a ideia eh vc propor PR's apenas nas branches script e packaging. A branch generated eh obtida rodando o ./generate e fazendo merge da branch packaging para a doc e as outras coisas de empacotamento. Se fugimos disso perdemos o control da geraçao do codigo e a sustentabilidade da ideia...

Eh possivel vc fazer os PR's dessa forma por favor? Vc conseguiria gerir a mesma branch generated assim ou precisa de mais algo para que seja possivel. Como eu tinha comentado, vc poderia alterar o script generate na branch script para corrigir o problema do XSD da receita, assim como fiz por examplo para corrigir o XSD da NFSe ABRASAF.

@mileo
Copy link
Author

mileo commented Jun 8, 2018

Vou dar uma olhada. no que vc falou. Estou validando o generateDS primeiro depois reescrevo o histórico

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants