Skip to content

[CLN] sale(_*): linting#4956

Open
Feyensv wants to merge 2 commits intomaster-ruff-anvfrom
master-sale-linting-vfe
Open

[CLN] sale(_*): linting#4956
Feyensv wants to merge 2 commits intomaster-ruff-anvfrom
master-sale-linting-vfe

Conversation

@Feyensv
Copy link

@Feyensv Feyensv commented Jan 20, 2026

No description provided.

@robodoo
Copy link

robodoo commented Jan 20, 2026

This PR targets the un-managed branch odoo-dev/odoo:master-ruff-anv, it needs to be retargeted before it can be merged.

@AntoineVDV AntoineVDV force-pushed the master-ruff-anv branch 2 times, most recently from 46f0650 to f475910 Compare February 9, 2026 11:42
Since the support for exclusions on optional products was dropped,
those two values were given but not used anymore.

Taking this opportunity to remove them as they trigger unused
method arguments linting checks
@Feyensv Feyensv force-pushed the master-sale-linting-vfe branch from cf880f6 to 9f80a46 Compare February 13, 2026 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants