Sincronização das branches Odoo-Brasil > OCA
154 views
Skip to first unread message
Renato Lima
Oct 21, 2015, 1:44:47 PM10/21/15
to openerp...@googlegroups.com
Olá Pessoal,
Eu fiz uma analise em todos os PR que foram feitos na branch 8.0 do Odoo-Brasil comparando com as branches da OCA, criei issues por módulos com listas dos PR que precisam ser integrados na OCA:
Com base nesta lista, alguns PR já foram integrados, mas existe outros que ainda falta ser integrados, o plano é finalizar os merges de todos os módulos na OCA/8.0 e depois integrar os PR listados nas issues.
Atualmente eu fiz modificações nos dados de demonstração e fiz alguns testes básico no módulo l10n_br_account_product (o mais complexos da localização) e com mais PRs para sincronizar, para nos auxiliar no processo de integração destes PR, estou trabalhando nisso e vou fazer o merge hoje desde módulo, desta forma vai ser fácil integrar na OCA/8.0 todas as mudanças que fizemos e em seguida integrar o PR.
@mileo, @danimaribeiro e @cmsalmeida,
Vocês podem integrar os PR destas listas no OCA/8/0:
Estes módulos já foram migrados na OCA/8.0 mas ainda tem alguns PR que não foram integrados. O trabalho não é complexo, mas se conseguirmos sinergia nisso, vamos ter as branches sincronizadas e vamos conseguir terminar rápido e seguir a P&D juntos.
É importante terminar isso porque no encontro do dia 7 a gente poderia aproveitar a reunião do pessoal para fazer com code sprint para dar inicio a migração para a versão 9.0.
Grande abraço a todos!
Renato Lima
Raphael Valyi
Oct 21, 2015, 11:52:05 PM10/21/15
to openerp...@googlegroups.com
Ola,
do meu lado eu dei uma geral no PEP8, mlhares de linhas, quero vomitar PEP8 agora...
Entao, todos PR's da 8.0 devem estar limpo no PEP8 agora...
...Menos o modulo l10n_br_account_product que e o proximo na lista dos merges e que ta engarafando todo...
Esse dai ta bravo, tinha muitos problemas principalmente na parte dos mapeamentos de nfe para txt e pysped...
Corrigi a metade, mas ainda sobra a outra metade:
Se vc tiver corragem de pegar um desses arquivo, avisa no PR que ta fazendo para evitar trabalho dobrado. E que teremos que ter isso limpo para entrar na 8.0 da OCA...
Os outros PR, podem ate nao rodar o PEP porque nao fix rebase deles (fiz do l10n_br_account_product porem e felizmente os commits da OCA nao deram conflito, ufa!)
Os testes no l10n_br_account_product nao estao rodando por causa do refator do Sylvain Legal no fiscal rules da OCA.
Com isso e provavel que a gente consegue extrair uma branch com esses PR e botar em algum lugar, odoo-brazil-team por examplo para mais pessoas testar e assm tb talvez ter os testes travis ate engulir o refator do fiscal rules (sou a favor de incrementar o numero de versao do modulo ao fazer isso alias).
Bom e isso ai pessoal. Ajuda sempre bem vinda. Aqui tamos conseguindo retomar um ritmo mais normal entao as coisas deveriam voltar a andar bem.
Abraco.
--
Você recebeu essa mensagem porque está inscrito no grupo "OdooBrasil / OpenERPBrasil.org" dos Grupos do Google.
Para cancelar inscrição nesse grupo e parar de receber e-mails dele, envie um e-mail para openerp-brasi...@googlegroups.com.
Para mais opções, acesse https://groups.google.com/d/optout.
Luis Felipe Miléo
Oct 22, 2015, 5:37:42 AM10/22/15
to openerp...@googlegroups.com
#raphael<br><br>Estou revisando e refatorando a parte do sped/nfe e algumas coisas do account_invoice.py<br><br>Até o fim da semana que vem vou fazer o merge de tudo isso.
- Luis Felipe Miléo
Gerência de Implementação
+55 11 4063-9872 (SP)
+55 21 4063-8872 (RJ)
mileo@kmee.com.br
Parceiro oficial no Brasil:
De: "Raphael Valyi" <rva...@gmail.com>
Para: openerp...@googlegroups.com
Enviadas: Quinta-feira, 22 de outubro de 2015 1:52:02
Assunto: Re: [openerpbrasil.org] Sincronização das branches Odoo-Brasil > OCA
Ola,do meu lado eu dei uma geral no PEP8, mlhares de linhas, quero vomitar PEP8 agora...Entao, todos PR's da 8.0 devem estar limpo no PEP8 agora.....Menos o modulo l10n_br_account_product que e o proximo na lista dos merges e que ta engarafando todo...Esse dai ta bravo, tinha muitos problemas principalmente na parte dos mapeamentos de nfe para txt e pysped...Corrigi a metade, mas ainda sobra a outra metade:Se vc tiver corragem de pegar um desses arquivo, avisa no PR que ta fazendo para evitar trabalho dobrado. E que teremos que ter isso limpo para entrar na 8.0 da OCA..
Raphael Valyi
Oct 22, 2015, 10:06:47 AM10/22/15
to openerp...@googlegroups.com
Ola Luis,
eu so peco uma coisa MUITO importante:
separe os commits que sao de PEP8 daqueles que sao de refator, senao vai ficar impossivel avaliar o refator. ok?
Essas correcoes do PEP8 que sobrou e um saco, temos que estar bem coordenados, porque isso impede de fazer o merge na OCA e um saco uma pessoa so resolver todos eles...
Raphael Valyi
Oct 22, 2015, 11:21:30 AM10/22/15
to openerp...@googlegroups.com
Luis,
peco tambem que procure contribuir os fixes do PEP8 antes do refator na ordem dos commits e que mandar uns PR na branch da Akretion logo que acertar uns arquivos.
Porque isso?
porque imagine que a gente tem alguma reserva nos refatores e que demoramos um pouco para acerta-los. A gente queria poder destravar os merges na OCA e o que bloca hoje e o PEP8.
Pelo menos queremos poder liberar logo uma branch no odoo-brasil-team com os tests de lint/pep8 habilitados. A gente poderia fazer isso ate engulir o refator da gestao do NCM. Seria bom ver se a gente consegue incluir um script de migracfao desse refator alias, que aqui temos uns 15 bancos para migrar...
Entao se a gente acerta o pep8 primeiro ta ok. Agora se vc fizer o refator primeiro, iremos ter que acertar o pep8 de qualquer forma de depos teremos o trabalho dobrado para fazer o rebase do seu refator que nao sera mais automatico...
Abraco,
2015-10-22 7:37 GMT-02:00 Luis Felipe Miléo <mi...@kmee.com.br>:
Luis Felipe Miléo
Oct 22, 2015, 11:33:34 AM10/22/15
to openerp...@googlegroups.com
Primeiro vou mandar o refactory, depois o pep em cima dele.
Raphael Valyi
Oct 22, 2015, 11:44:59 AM10/22/15
to openerp...@googlegroups.com
@Luis,
o risco fca com vc entao...
Tem mais uma consideracao: e o front-port dos PR's da odoo-brasil-team. Fica mas facil inclui-los antes de refatorar...
Mas enfim se o refator for bom e nao muito arriscado vamos incluir sem problema.
Abraco.
Danimar Ribeiro
Oct 22, 2015, 4:57:08 PM10/22/15
to openerp...@googlegroups.com
Na verdade este módulo ficou muito grande e praticamente impossível de ver o antes e depois dele,
acho que as mudanças de pep juntamente com a migração no mesmo PR é complicado.
Seria bom que vocês integrassem o PR logo, já faz muito tempo que estamos esperando pelos testes dele.
Raphael Valyi
Oct 22, 2015, 6:22:12 PM10/22/15
to openerp...@googlegroups.com
@Danimar,
o que podemos fazer e botar uma outra work branch no team odoo-brasil-team que integra ele, talvez hoje ainda.
Agora fazer o merge antes de resolver o PEP8 nao rola. Como expliquei a gente deve aplicar esses regras basicas da OCA.
Se todos teams pedirem execoes para nao respeitar as regras da OCA, ai logo a OCA vai estar com a metade dos modulos zoados e depois sera impossivel educar o mercado para valorisar um minimo de qualidade. a OCA nao seria diferente da Odoo SA, seria a mesma bagunca e ninguem iria mais accreditar em nada.
Je falei minha opinao sobre a questao: a OCA establece regras que ficam legal na Europa para empresa que vende projeto Odoo de 10 milhoes de euros e um pouco inadaptado no mercado daqui onde nego acha eu qualquer 5k resolve se nao for muito (se nao me engano o ultimo projeto da C2c e de 15 milhoes de euros mesmo). Mas isso e um problema do Brasil, nao podemos pedir para a OCA de se adaptar para a gente. Agora sim, talvez nao ha tenta pressa de botar mais modulos do que do core na OCA, isso sim. Se vc tivesse acompanhado as listas da OCA teria visto que isso ja gerou muto debate com muita gente reclamando desse rigor mas a C2C e SFL foram bem rigidos nisso. Como sao os pilares da OCA...
Tambem nao adianta reclamar e nao ajudar. Sei que e algo chato, mas se alguem tivesse encarado o PEP8 antes certamente teria levado muito menos tempo neh...
Vc consegue rastreiar as funcionalidades pelos commits. Por mim umas 6 horas de trabalho de da para matar o PEP8 que faltou. Se o Luis quer fazer melhor ainda.
@Luis vc comentou que quer refatorar. o sped/serializer/txt.py, vc pretende mexer nele tb? Senao eu mato o PEP8 dele em paralelo, por favor avise.
Abraco.
Raphael Valyi
Oct 22, 2015, 6:32:49 PM10/22/15
to openerp...@googlegroups.com
@Danimar,
daqui do Brasil aquilo pode parecer inimagininavel, Mas aquilo de educar mercado para exigir qualidade da OCA comeca a funcionar. Em projetos grandes na Franca, a gente realmente vende para o cliente que 80% dos modulos dele serao incluidos na OCA. Acho que a CampToCamp faz gual tb. A gente consegue passar para o cliente a ideia que assim ele tera menos bugs e modulos mais robustos o dia que vai querer migrar de versao ou para ter a liberdade de poder trocar de integrador...
O Odoo comecou com padroes de qualidade extremamente baixo, tipo nem professional na hora do TinyERP. Mas o lance e que ninguem tinha feito um ERP open source melhor mesmo assim... Entao temos que fazer um trabalho de educacao do mercado gigantesco. Quando vc ve a bagunca que e esse appstore e que isso e o modelo que Odoo SA promove, da para entender que isso so vai ir para frente so se a OCA for capaz de propor um contra-modelo. E nois aqui no Brasil estamos totalmente nas margens desse eco-sistema, mas nem por isso podemos pedir para a OCA nivelar por baixo.
Danimar Ribeiro
Oct 23, 2015, 8:23:25 AM10/23/15
to openerp...@googlegroups.com
A gente está esperando vocês, faz um bom tempo que vocês vem falando para nós que ta quase pronto, que os testes estavam saindo e iam publicar.
Eu não mexi ainda no código do PR l10n_br_account_product pois estava esperando vcs publicarem, não da pra mandar email dizendo que ja vai publicar e depois não publica, eu fiquei esperando várias vezes.
Acho que a migração da localização na OCA, já está muito atrasada, então recomendo esquecer um pouco da OCA e pensar por 1 minuto na melhor forma de trabalhar na localização, não acredito que eles vão reclamar por ficar 1 semana com build falhando, por causa de pep8. Ninguém está pedindo para não respeitar as regras da OCA, estou pedindo que se aceite temporariamente para destrancar o trabalho em outros módulos e o trabalho para outras pessoas. Desta forma vamos terminar mais rapidamente.
Estou bem ciente das regras da OCA, mas também sei que todo projeto existem exceções e se conversar com as pessoas da OCA elas são bem compreensíveis aos fatos.
Raphael Valyi
Oct 23, 2015, 9:38:33 AM10/23/15
to openerp...@googlegroups.com
'@Danimar
A gente ja explicou que passamos por um problema interno. Duvido se vc teria feito melhor se tivesse passado por isso. Lamento mas fatores humanos acontecem no open source. Olha onde foi parar o ReiserFS... Agora a gente faz a OCA exactamente para trazer mais seguranca nesse mundo e com tudo que a gente faz para a OCA existr vc nem pode criticar tb.
Se nao se preocupa com a OCA, ninguem impede vc de usar uma outra branch num cliente, assim como fazemos faz meses entao nao vem com essa. So talvez vc tem que ser cauteloso antes de recomendar para alguem usar outra branch com o trabalho de migracao que poderia ter.
Agora, a respeito da OCA tem regras e nao sou nem eu nem vc que decide quais sao (a nao ser votando na hora das decisoes). Se vc nao concorda com as regras da OCA, tinha que convencer os caras la 2 anos antras quando isso foi decidido...
Todo projetos da OCA repeitam esse processo.
Pega por examplo o projeto HR, olha quantos modulos ainda nao foram migrados para a v8:
Os caras nao estao cedendo a alguem que faria pressaozinha para fazer os merges sem acertar a qualidade primeiro nao e nao aceitarao que a gente faca aqui.
Quer acelerar os merges? Corrige entao alguns dos 5 arquivos do pep8 que falta. So fale quais vc pega no PR para nao fazer em dobro. Sao os uns 5 arquivos que enchem o saco agora, da para paralelisar. Qualquer pessoa com um ano de Python pode fazer isso nao ha porque ser so a gente.
Eu tambem pensei que teriamos os testes antes. Mas o Renato ficou super ocupado, acho que ele ja fez demais no projeto para a gente ter direito de reclamar.
Na verdade, tinhamos testes fiscais bem completos ao nivel dos modulos sale e purchase no inicio do ano por isso que pensei que seria rapido. Mas e pesado extrai-los e rebota-los ao nivel so do modulo account...
Mas testes sao bem vindos tb neh.
Sao 390 falhas PEP8 ainda. Corrigi centenas esses dias, Pegue algumas e mande um PR entao por favor.
Danimar Ribeiro
Oct 23, 2015, 10:35:50 AM10/23/15
to openerp...@googlegroups.com
Porque vocês não pediram ajuda antes nesse PR?
Faz tempo que estamos esperando, vocês falaram que já estava quase pronto.
Na verdade temos direito de reclamar sim, não faltou vontade nossa de ajudar, mas sempre as respostas foram, já estamos terminando e vamos publicar.
Raphael Valyi
Oct 23, 2015, 10:54:13 AM10/23/15
to openerp...@googlegroups.com
Danimar,
eu mergulhei nos detalhes do PEP8 so recentemente nesse PR infelizemente porque so desde recentemente que eu consigo trabalhar o bastante para isso.
Eu queria testes ao nivel do codigo da logica aplicativa e dos objetos persistentes para evitar erros que corompessem o bancos que sao os mais chatos de corigir depois. Confesso que ainda temos poucos, esperei por eles antes de pegar o PEP-8 desse codigo aplicativo.
Agora vejo que nao tem como: a curto prazo nao teremos testes dos mapeamentos XML e txt das notas. Vamos ter que accrescentar, mas a curto prazo nao teremos, na ha como. Agora sou a favor da gente botar alguns antes de pegar a migracao da v9 da contabilidade ja que de qualquer jeito vai engarafar na OCA.
Entao nao tem como esses erros de PEP8 desse codigo legacy de mapeamento vamos ter que corrigir tomando muito cuidado e ajuda e bem vinda sim.
Consigo matar a metade do que sobrou ate segunda. Entao se vc ou o Luis pegar a outra metade, logo temos esse merge. Ontem eu acertei o PEP8 dos outros PR's ja entao o garagalho e so esse mesmo, vale a pena dar uma forca.
Do restante, a menos se tiver instrucao contraria, todos PR's na OCA que nao estao finalizados tem ajuda bem vinda, sempre foi assim.
Novamente a situacao e essa porque por politica comercial a Odoo SA preferiu investir nesse appstore em vez de empacotar os modulos no pypi como o padrao. Se fosse, cada modulo teria um repo, a gente faria so pip install do requirements.txt para instalar numa boa. A OCA esta trabalhando nisso e vc pode ver que sou um dos caras que sempre apontou esse ponto. Infelizmente hoje temos que lidar com essas branches multi-modulos e essa floresta de PR...
Luis Felipe Miléo
Oct 23, 2015, 1:00:02 PM10/23/15
to openerp...@googlegroups.com
Justo @danimar, <br><br>Vejo q agora se tem pressa pq já podemos mandar PR da v9 pq a branch v9 foi criada automaticamente.<br><br>[]s
- Luis Felipe Miléo
Gerência de Implementação
+55 11 4063-9872 (SP)
+55 21 4063-8872 (RJ)
mileo@kmee.com.br
Parceiro oficial no Brasil:
Renato Lima
Oct 23, 2015, 2:19:13 PM10/23/15
to openerp...@googlegroups.com
@Luis,
Também não é justo criticar a situação da 8.0 e querer fazer a mesma coisa na 9.0, o objetivo é que na 9.0 não tenhamos o mesmo problema que houve na 8.0 e a forma de evitarmos isso e integrar os PRs antes e do nosso lado vamos procurar resolver boa parte do problema.
@Danimar,
Realmente eu levei tempo para concluir os testes, (amanhã nós aqui na Akretion vamos pegar pesado no l10n_br_account_product e vamos fazer o push na OCA/8.0), mas quando abri este tópico, comentei que existe PR que já dar para integrar no OCA/8.0 e não precisa esperar o l10n_br_account_product, já pode ser feito agora, tem alguns importantes no l10n_br_account:
https://github.com/OCA/l10n-brazil/issues/190 - l10n_br_account
https://github.com/OCA/l10n-brazil/issues/195 - l10n_br_crm e l10n_br_crm_zip
https://github.com/OCA/l10n-brazil/issues/197 - l10n_br_zip
Até tentei falar com você esta semana para organizarmos estes trabalhos, mas você tinha me falado que esta semana não estava trabalhando, se tiver online amanhã podemos tocar uma ideia para trabalhar juntos nisso.
Reply all
Reply to author
Forward
0 new messages