Skip to content

[20.0][IMP] update dotfiles - #1030

Open
carlosdauden wants to merge 2 commits into
OCA:20.0from
Tecnativa:20.0-imp-update-dotfiles-v1.48
Open

carlosdauden wants to merge 2 commits into
OCA:20.0from
Tecnativa:20.0-imp-update-dotfiles-v1.48

Conversation

@carlosdauden

Copy link
Copy Markdown

Update the 20.0 branch to oca-addons-repo-template v1.48 (copier update --vcs-ref=v1.48).

The main effect on 20.0 is OCA/oca-addons-repo-template#376: ruff ignores RUF012 (mutable-class-default) for Odoo >= 20.0, as Odoo core does in its ruff.toml. The pre-commit rev of ruff (0.16.8) enables RUF012 by default, so with v1.45 every _inherit = ["a", "b"] in a migrated module fails pre-commit unless it carries # noqa: RUF012. With this update lists keep passing, as in Odoo core.

Other template changes since v1.45: GitHub workflows hardened and actions pinned to recent versions (OCA/oca-addons-repo-template#374), and a checklog-odoo ignore for a bootstrap warning when html_editor is not installed (OCA/oca-addons-repo-template#378).

Same update as OCA/server-tools#3836 (renovate).

A second commit removes the two # noqa: RUF012 comments that become unused (RUF100) with this update; ruff check --fix in pre-commit removes them automatically.

ping @sergio-teruel @carlos-lopez-tecnativa @juancarlosonate-tecnativa

Update the oca-addons-repo-template to v1.48. The main effect on 20.0 is that ruff ignores RUF012 (mutable-class-default), like Odoo core does, so `_inherit` lists pass ruff 0.16 without `# noqa`. It also pins the GitHub actions to recent versions and adds a checklog-odoo ignore for the bootstrap color warning when html_editor is not installed.
Remove the `# noqa: RUF012` comments, now unused (RUF100) because the template v1.48 ignores RUF012 on 20.0.
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @guewen, @sbidoul,
some modules you are maintaining are being modified, check this out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants