Closed justincpresley closed 2 years ago
isinstance
one). Thank you.if a > 0:
return "Positive"
else:
return "Non-Positive"
is better than
if a > 0:
return "Positive"
return "Non-Positive"
because "Positive" and "Non-Positive" are actually belonging to the same layer -- there is no reason to let them have different indents just because of the return
.
I don't think I will merge this PR. But I will consider fix some of them.
Found these places via the tool codefactor. There are other issues that it detects, but I left those open as there could be different ways to solve them.