Skip to content

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