[IMP] consider unsafe fixes for UP008 as safe

[trisdoan]

Context

• Take over this improvement: [IMP] When migrating code, automatically apply PEP 3135 odoo-module-migrator#77
• Instead of implementing in odoo-module-migrator (not always installed), do it directly with Ruff pre-commit hook

This change

• As this rule is taken into account in Ruff, simply applying its fix when with extend-safe-fixes

[sbidoul]

Suggested change

	{%- if odoo_version >= 11.0 %}
	extend-safe-fixes = ["UP008"]
	{%- endif %}
	extend-safe-fixes = ["UP008"]

Since we don't support Odoo 10 anymore, and ruff does not support python 2 anyway no need for this version test.

[sbidoul]

Do we know if ruff UP008 preserves exoctic uses of super that we have in Odoo? Things like super(Class, self.filtered(...))._method().

[trisdoan]

Do we know if ruff UP008 preserves exoctic uses of super that we have in Odoo? Things like super(Class, self.filtered(...))._method().

Hello @sbidoul, yes it does. As long as the second argument is NOT equivalent to the first argument of the enclosing method, due to this method

For example, it does not impact this case

return super(
            QueueJob,
            self.with_context(mail_create_nolog=True, mail_create_nosubscribe=True),
        ).create(vals_list)

[sbidoul]

Great. Let me know when you have updated the PR to remove the version test.

[OCA-git-bot]

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). 🤖