erpbrasil / erpbrasil.base

MIT License
14 stars 15 forks source link

[UPD+FIX] Atualizando as bibliotecas do Pre-commit e corrigido conflito entre validações #46

Open mbcosta opened 8 months ago

mbcosta commented 8 months ago

Ao verificar o problema dos PRs https://github.com/erpbrasil/erpbrasil.base/pull/44 e https://github.com/erpbrasil/erpbrasil.base/pull/45 acabei rodando o comando do pre-commit localmente para manter o padrão do código, acabei vendo que estava retornando um erro E203 e procurei atualizar as bibliotecas para ver se resolvia, mas na verdade existe um conflito entre as bibliotecas Black e Flake8

https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#slices

This behaviour may raise E203 whitespace before ':' warnings in style guide enforcement tools like Flake8. Since E203 is not PEP 8 compliant, you should tell Flake8 to ignore these warnings.

https://www.flake8rules.com/rules/E203.html

https://flake8.pycqa.org/en/3.1.1/user/ignoring-errors.html

Então nesse PR estou atualizando as Libs, corrigindo esse conflito e ao rodar o comando pre-commit acaba refatorando algumas partes do código, com isso o pre-commit passou a rodar sem erros pelo menos localmente, é preciso verificar se vai precisar mais alterações aqui no github.

cc @renatonlima @rvalyi @marcelsavegnago @mileo

mbcosta commented 8 months ago

Incluído um commit sobre o erro

https://github.com/erpbrasil/erpbrasil.base/actions/runs/7618270413/job/20749074292?pr=46#step:5:149 ( readme: line 3) broken https://api.travis-ci.org/erpbrasil/erpbrasil.base.svg?branch=master - 404 Client Error: NOT FOUND for url: https://api.travis-ci.org/erpbrasil/erpbrasil.base.svg?branch=master build finished with problems, 1 warning.

Pela mensagem a Imagem não foi encontrada, deveria ou faltou ser incluída?

antoniospneto commented 8 months ago

Incluído um commit sobre o erro

https://github.com/erpbrasil/erpbrasil.base/actions/runs/7618270413/job/20749074292?pr=46#step:5:149 ( readme: line 3) broken https://api.travis-ci.org/erpbrasil/erpbrasil.base.svg?branch=master - 404 Client Error: NOT FOUND for url: https://api.travis-ci.org/erpbrasil/erpbrasil.base.svg?branch=master build finished with problems, 1 warning.

Pela mensagem a Imagem não foi encontrada, deveria ou faltou ser incluída?

acho que o certo é remover mesmo, o travis não é mais utilizado aqui