[15.0][IMP] update copier from template + fixes to pass pre-commit in payroll module

OCA / payroll

Odoo modules for payroll management

GNU Affero General Public License v3.0

79 stars 120 forks source link

[15.0][IMP] update copier from template + fixes to pass pre-commit in payroll module #147

Closed AaronHForgeFlow closed 8 months ago

AaronHForgeFlow commented 8 months ago

@nimarosa do you think this is fine?

nimarosa commented 8 months ago

Hello @AaronHForgeFlow. Here you are updating the copier template but also changing files in payroll module. The PR updating the copier template should be unique and do not change other modules.

This is to maintain order and traceability of changes in the modules.

If you want to update copier template please do only this in this PR. Then the t-esc-options to t-options change, should be in another PR.

AaronHForgeFlow commented 8 months ago

Hi @nimarosa, I was forced to do so because pre-commit was failing after the update. When that happens I think we can fix the CI in a separate commit by module. For example: https://github.com/OCA/stock-logistics-warehouse/pull/1877/commits

nimarosa commented 8 months ago

Fair enough. You are right.

/ocabot merge nobump

OCA-git-bot commented 8 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 15.0-ocabot-merge-pr-147-by-nimarosa-bump-nobump, awaiting test results.

OCA-git-bot commented 8 months ago

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

AaronHForgeFlow commented 8 months ago

@nimarosa Thank you!