OCA / account-financial-tools

Odoo Accountant Financial Tools and Utils
GNU Affero General Public License v3.0
305 stars 748 forks source link

[15.0] dotfiles update needs manual intervention #1758

Closed etobella closed 10 months ago

etobella commented 10 months ago

Dear maintainer,

After updating the dotfiles, pre-commit run -a fails in a manner that cannot be resolved automatically.

Can you please have a look, fix and merge?

Thanks,

matteoopenf commented 10 months ago

@pedrobaeza is possible merge to fix pre-commit here https://github.com/OCA/account-financial-tools/pull/1635? thank you

pedrobaeza commented 10 months ago

@matteoopenf you need to supersede the PR and fix remaining pre-commit stuff.

matteoopenf commented 10 months ago

@matteoopenf you need to supersede the PR and fix remaining pre-commit stuff.

ok thank you for the help I will do

matteoopenf commented 10 months ago

@etobella have you any idea to fix?

etobella commented 10 months ago

Don't worry. I will do it directly here

/ocabot merge patch

OCA-git-bot commented 10 months ago

This PR looks fantastic, let's merge it! Prepared branch 15.0-ocabot-merge-pr-1758-by-etobella-bump-patch, awaiting test results.

OCA-git-bot commented 10 months ago

@etobella your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-1758-by-etobella-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.

matteoopenf commented 10 months ago

Don't worry. I will do it directly here

/ocabot merge patch

thank you so much for the help

OCA-git-bot commented 10 months ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

pedrobaeza commented 10 months ago

/ocabot merge patch

OCA-git-bot commented 10 months ago

This PR looks fantastic, let's merge it! Prepared branch 15.0-ocabot-merge-pr-1758-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 10 months ago

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