[14.0][REF] Atualização do repositório (copier update)

[antoniospneto]

Atualização do repositório com o "copier update"

[OCA-git-bot]

Hi @felipemotter, @mileo, @felipezago, @ygcarvalh, @hendixcosta, @renatonlima, @AndreMarcos, @rvalyi, @DiegoParadeda, @lfdivino, @luismalta, @mbcosta, @marcelsavegnago, @gabrielcardoso21,
some modules you are maintaining are being modified, check this out!

[rvalyi]

Na v16, tem algumas dessas regras novas que eu acabei burlando para conseguir migrar e sem dar muito diff com a v14. Se vc arrumar agora na v14, vai valer a pena repercutir esses refatores na v16 tb. Por exemplo o merge das classes de ncm (sendo arquivos pequenos eu acho tudo bem fazer esse merge). Segue a lista das exeçoes que fizemos na v16 com a branch 16.0 atual:

❯ grep -r "pylint:" . | grep -v file-not-used

./l10n_br_account/models/account_move_line.py:# pylint: disable=api-one-deprecated
./l10n_br_fiscal_dfe/tests/test_dfe.py:# pylint: disable=line-too-long
./l10n_br_sale/models/sale_order.py:    # pylint: disable=except-pass
./l10n_br_coa_generic/data/l10n_br_coa_generic_template_post.xml:<!-- pylint:disable=duplicate-xml-record-id -->
./l10n_br_fiscal/models/tax_definition_company.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_fiscal/models/tax_definition_benefit.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_fiscal/models/ncm_cest.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_fiscal/models/ncm_nbm.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_fiscal/models/ncm_tax_pis_cofins.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_fiscal/models/tax_definition_cfop.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_fiscal/models/tax_definition_icms.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_fiscal/models/tax_definition_operation_line.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_fiscal/models/tax_definition_partner_profile.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_coa_simple/data/l10n_br_coa_simple_template_post.xml:<!-- pylint:disable=duplicate-xml-record-id -->
./l10n_br_nfe/models/document.py:                        self.env.cr.commit()  # pylint: disable=invalid-commit
./l10n_br_nfe/tests/test_nfe_dfe.py:# pylint: disable=line-too-long
./l10n_br_nfe/tests/test_nfe_mde.py:# pylint: disable=line-too-long
./l10n_br_cnpj_search/models/serpro_webservice.py:# pylint: disable=R8180
./l10n_br_ie_search/models/sefaz_webservice.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_ie_search/models/sintegra_webservice.py:# pylint: disable=consider-merging-classes-inherited
./l10n_br_cnab_structure/__manifest__.py:# pylint: disable=pointless-statement
./l10n_br_nfse_focus/models/document.py:            response = requests.request(  # pylint: disable=external-request-timeout

[antoniospneto]

Pessoal deixei a PR bem simples, agora não tá mais as alterações nos arquivos README.
é apenas os lint no código python, maior parte fiz backport da 16.

[antoniospneto]

Observação importante.

Na configuração do pre-commit estou forçando a permitr apenas a versão do python 3.11
Pois a nova versão do python 3.12 é imcompativel com a versão de algumas ferramentas usadas aqui.
Porém isso vai obrigar o desenvolvedor a ter instalado o python 3.11 em sua maquina local.
Quanto ao CI é tranquilo pois já está travado para usar o python 3.11.

.pre-commit-config.yaml:

...
default_language_version:
  python: python3.11
...

Pra quem precisar instalar o python 3.11: https://gist.github.com/antoniospneto/ee1b6a55748ceba77968e5c6ce3f9ea0

[antoniospneto]

Separei os commits em dois, para facilitar a revisão e tbm o port:

O primeiro contém apenas os arquivos de configuração alterado pelo "copier update"
O segundo é as alteraçõoes para passar no pre-commit.

Aproveitei e já fiz o port de umas coisas que estavam berrando no pre-commit da v.15.0

[rvalyi]

OK para mim, porem eu acho que temos que avaliar a possibilidade de merge de #3337 e talvez outros antes para não aumentar o trabalho de rebase do @mbcosta

EDIT: eu atei botei como blocked pois tem alguns PR's que temos que avaliar antes, como:

• [14.0][REF] l10n_br_account_payment_order, l10n_br_account_payment_brcobranca: Unificando os Códigos CNAB em um único objeto #3337
• [14.0][REF] l10n_br_sale_stock: Extraction referent the creation of the module sale_stock_picking_invoicing #2955
• [14.0][FIX] l10n_br_sale: Fatura criada a partir do Pedido de Vendas deve usar o Partner do campo partner_invoice_id #2864

[rvalyi]

@antoniospneto rola um rebase? Agora que entrou pelo menos o refator do CNAB code eu tirei o blocked, ja ta mais suave entrar....

[antoniospneto]

@antoniospneto rola um rebase? Agora que entrou pelo menos o refator do CNAB code eu tirei o blocked, ja ta mais suave entrar....

feito!

[marcelsavegnago]

LGTM

[rvalyi]

dei um merge de #3243, se puder dar rebase...

[antoniospneto]

/ocabot merge patch

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-3345-by-antoniospneto-bump-patch, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 4ea379c. Thanks a lot for contributing to OCA. ❤️