style: Fix superfluous-else rules for raise, continue and break (RET506, RET507, RET508) #4565
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Following #4459, where when there was an
if
with a return, the nextelif:
orelse:
could be flattened with one less nested level, this PR addresses three similar rules, where instead of havingreturn
, there isRET506
for araise
,RET507
for continue in a loop, andRET508
for abreak
in a loop.There are only 35 changes files, and quite simple, but in a couple of files, there was a lot of nested conditions/long nested conditions that became flattened and got reformatted with fewer lines. So the line change count is bigger than it appears.