Introduce flake8 return

Based on !2281 (merged)

I've made one commit per rule. I'm not sure I agree with:

• RET502
• RET503.

Please tell me what you think, I can easily remove them.

changed milestone to %Ruff linting/formatting

added P3 label

assigned to @yschneider

I like the elif => if conversion but not the return None

Agreed, we can skip on RET502 and RET503

requested review from @erouchet

added 20 commits

• 19ea4b94...c76fdeeb - 15 commits from branch master
• ddd4f9c2 - Comply with RET501
• e190b611 - Comply with RET504
• d7c2bb74 - Comply with RET505
• 23e7d6a8 - Comply with RET506
• 8facf5bc - Comply with flake8-return

Compare with previous version

resolved all threads

added 7 commits

• 8facf5bc...149110fd - 2 commits from branch master
• 114182bb - Comply with RET501
• 9360898f - Comply with RET504
• 2f790e93 - Comply with RET505
• e648d5e4 - Comply with RET506
• e7787132 - Comply with flake8-return

Compare with previous version

enabled an automatic merge when the pipeline for e7787132 succeeds

approved this merge request

merged

mentioned in issue #1744 (closed)

mentioned in issue #1745 (closed)