[14.0][REF] Atualização da bibliotecas do Pre-commit: Remoção de Linhas Vazias #2749
Conversation
…ion of pre-commit.
…version of pre-commit.
…ry for the new version of pre-commit.
…r the new version of pre-commit.
…ew version of pre-commit.
…sion of pre-commit.
…sion of pre-commit.
…on of pre-commit.
…rsion of pre-commit.
…version of pre-commit.
…new version of pre-commit.
…on of pre-commit.
…sion of pre-commit.
… new version of pre-commit.
…sion of pre-commit.
… new version of pre-commit.
…on of pre-commit.
…sion of pre-commit.
…he new version of pre-commit.
…e new version of pre-commit.
…ersion of pre-commit.
…w version of pre-commit.
… the new version of pre-commit.
… version of pre-commit.
…rsion of pre-commit.
|
Hi @ygcarvalh, @rvalyi, @luismalta, @felipezago, @lfdivino, @antoniospneto, @gabrielcardoso21, @mileo, @marcelsavegnago, @hendixcosta, @felipemotter, @renatonlima, |
|
lembrando que a vantagem de fazer essas coisas agora é facilitar os futuros cherry-picks entre as branches 14 e 16 por examplo. |
|
This PR looks fantastic, let's merge it! |
|
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2749-by-rvalyi-bump-patch. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
|
Valeu @rvalyi @marcelsavegnago pela revisão, pelo o que entendi o PyLint na versão recente não aceita a primeira linha vazia em métodos def ou if ou na definição da classe, bom isso deve melhorar a visualização e reduzir linhas no código |
|
Durante o merge ocorrem erros de Pre-Commit que não aprecem aqui https://github.com/OCA/l10n-brazil/actions/runs/6659297375/job/18098070274#step:7:993
Estranho a alteração coloca uma quantidade a mais de aspas " que não parece certa |
capaz que se trata desse problema que outras pessoas da OCA identificaram OCA/oca-addons-repo-template#214 |
|
/ocabot merge patch |
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at 376f0e7. Thanks a lot for contributing to OCA. ❤️ |
Remove Empty Lines, validation made in the recent pre-commit librarys.
Remoção de Linhas Vazias, apesar de alterar diversos arquivos e módulos não afeta nenhum código.
Eu estou vendo a possibilidade de atualizar as bibliotecas do Pre-Commit no PR #2747 ( isso ainda está em debate por existir uma forma automatizada de fazer isso, mas essa alteração já antecipa o que vai precisar ser feito quando houver a atualização ), essa validação está sendo feita nas recentes versões das bibliotecas do pre-commit https://github.com/OCA/l10n-brazil/actions/runs/6643339050/job/18050180768?pr=2747#step:7:63
cc @rvalyi @renatonlima @marcelsavegnago @mileo