l10n-brazil
l10n-brazil copied to clipboard
OCA
[12.0] switch to gh actions
Hi @marcelsavegnago, @renatonlima, some modules you are maintaining are being modified, check this out!
Visto que os PRs:
https://github.com/starkbank/febraban-python/pulls
Por exemplo o do Extrato bancário: https://github.com/starkbank/febraban-python/pull/26 não foram nem revisados nos últimos dois anos, a solução vai ser criar um fork e subir no pypi, não tem jeito....
@netosjb @felipemotter vocês estão gerando o arquivo CNAB naquele PR que vcs fizeram? Estão usando essa lib?
Sei lá isso ou arrumar um jeito de desabilitar a instalação no GitHub Actions vc que sabe... Mas eu insisto é importante ajeitar o GitHub Action. Senão os testes na 12 vão acabar quebrando e ninguém mais vai manter.
Visto que os PRs:
https://github.com/starkbank/febraban-python/pulls
Por exemplo o do Extrato bancário: starkbank/febraban-python#26 não foram nem revisados nos últimos dois anos, a solução vai ser criar um fork e subir no pypi, não tem jeito....
@netosjb @felipemotter vocês estão gerando o arquivo CNAB naquele PR que vcs fizeram? Estão usando essa lib?
não estamos usando, o módulo que fizemos não utiliza lib externa
Sei lá isso ou arrumar um jeito de desabilitar a instalação no GitHub Actions vc que sabe... Mas eu insisto é importante ajeitar o GitHub Action. Senão os testes na 12 vão acabar quebrando e ninguém mais vai manter.
Eu acho a lib bem boa e fácil de manter, se não divergirmos muito do fork deles não vai dar tanto trabalho.
Visto que os PRs: https://github.com/starkbank/febraban-python/pulls Por exemplo o do Extrato bancário: starkbank/febraban-python#26 não foram nem revisados nos últimos dois anos, a solução vai ser criar um fork e subir no pypi, não tem jeito.... @netosjb @felipemotter vocês estão gerando o arquivo CNAB naquele PR que vcs fizeram? Estão usando essa lib?
não estamos usando, o módulo que fizemos não utiliza lib externa
Algum motivo pra fazer na mão?
Sei lá isso ou arrumar um jeito de desabilitar a instalação no GitHub Actions vc que sabe... Mas eu insisto é importante ajeitar o GitHub Action. Senão os testes na 12 vão acabar quebrando e ninguém mais vai manter.
Eu acho a lib bem boa e fácil de manter, se não divergirmos muito do fork deles não vai dar tanto trabalho.
Visto que os PRs: https://github.com/starkbank/febraban-python/pulls Por exemplo o do Extrato bancário: starkbank/febraban-python#26 não foram nem revisados nos últimos dois anos, a solução vai ser criar um fork e subir no pypi, não tem jeito.... @netosjb @felipemotter vocês estão gerando o arquivo CNAB naquele PR que vcs fizeram? Estão usando essa lib?
não estamos usando, o módulo que fizemos não utiliza lib externa
Algum motivo pra fazer na mão?
Nossa proposta foi fazer algo mais "dinâmico" o parse da estrutura cnab é feita nas views do Odoo, depois da uma olhada aqui: https://www.youtube.com/watch?v=ljommELunlA
pessoal, eu consegui ter esse Github Actions rodando desabilitando o teste do modulo l10n_br_account_bank_statement_import_cnab por enquanto (ver ultimo commit).
vale a pena notar que no Travis a gente ja teve que desabilitar esses modulos, todos da KMEE... https://github.com/OCA/l10n-brazil/blob/12.0/.travis.yml#L47 e que davam problema.
Depois desse merge daria para re-habilitar os testes do payment_pagseguro com essa PR https://github.com/OCA/l10n-brazil/pull/2209
On my way to merge this fine PR! Prepared branch 12.0-ocabot-merge-pr-2130-by-mileo-bump-patch, awaiting test results.
Congratulations, your PR was merged at 93a3d3346b50201c3f30bc7331088216aa26539c. Thanks a lot for contributing to OCA. ❤️