Skip to content

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

Merged
AntoineVDV merged 2 commits intomaster-ruff-anvfrom
master-sale-linting-vfe
Mar 10, 2026
Merged

[CLN] sale(_*): linting#4956
AntoineVDV merged 2 commits intomaster-ruff-anvfrom
master-sale-linting-vfe

Conversation

@Feyensv
Copy link
Copy Markdown

@Feyensv Feyensv commented Jan 20, 2026

No description provided.

@robodoo
Copy link
Copy Markdown

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
@Feyensv Feyensv force-pushed the master-sale-linting-vfe branch 2 times, most recently from 9f80a46 to 54e5cac Compare February 13, 2026 14:41
@Feyensv Feyensv force-pushed the master-sale-linting-vfe branch from 54e5cac to fa3b596 Compare February 17, 2026 17:15
@AntoineVDV AntoineVDV force-pushed the master-ruff-anv branch 3 times, most recently from c236ea9 to 74a945e Compare March 5, 2026 17:11
@Feyensv Feyensv force-pushed the master-sale-linting-vfe branch from fa3b596 to d0f2699 Compare March 10, 2026 11:17
Feyensv added 2 commits March 10, 2026 13:22
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 2 times, most recently from a13f173 to 64011d9 Compare March 10, 2026 13:37
@AntoineVDV AntoineVDV merged commit ab5baf3 into master-ruff-anv Mar 10, 2026
@AntoineVDV AntoineVDV deleted the master-sale-linting-vfe branch March 10, 2026 14:49
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.

3 participants