Fix some issues found by lgtm.com by misery · Pull Request #64 · techtonik/python-patch
-
Notifications
You must be signed in to change notification settings - Fork 66
Open
wants to merge 1 commit into
Conversation
- This import of module shutil is redundant, as it was previously imported on line 35.
- Testing for None should use the 'is' operator.
- The value assigned to local variable 'hno' is never used.
* This import of module shutil is redundant, as it was previously imported on line 35. * Testing for None should use the 'is' operator. * The value assigned to local variable 'hno' is never used.
This was referenced
lgtm.com is right about shutil, but I've heard from Fedora users that shutil has some permission problems. I need to ask them for testing as well.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 participants
