Looks like recent change in pylint trigger this incorrect report:
drenv/commands.py:234:28: E0606: Possibly using variable
'input_view' before assignment (possibly-used-before-assignment)
This cannot happen since we don't register proc.stdin if input is None, so when we reach this block input_view is assigned. However disabling the check risk missing a real issue in that block.
Lets change the code so pylint can understand it better. This also make it easier to understand for humans. The cost is negligible, adding 2 temporary variables even when they are never used.
Looks like recent change in pylint trigger this incorrect report:
This cannot happen since we don't register proc.stdin if input is None, so when we reach this block input_view is assigned. However disabling the check risk missing a real issue in that block.
Lets change the code so pylint can understand it better. This also make it easier to understand for humans. The cost is negligible, adding 2 temporary variables even when they are never used.