-
-
Notifications
You must be signed in to change notification settings - Fork 274
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[misc-] fix minor name definition errors caught by pyflakes #2428
base: develop
Are you sure you want to change the base?
Conversation
replayCommand() is unused, but fixing it silences linter warnings.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean that pyflakes only found these 10 issues and they were all minor? Or that, of the numerous issues pyflakes found, these are the 10 minor ones that could be easily fixed?
@@ -179,10 +179,6 @@ def name(self): | |||
'Filename without any extensions. Not the same as pathlib.Path.' | |||
return self.base_stem | |||
|
|||
@lru_cache() | |||
def stat(self, force=False): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing this makes me nervous. It should still work, as members that aren't defined explicitly fall back to self._path, but the @lru_cache will make this much faster for DirSheets which would otherwise stat the same file repeatedly. How can pyflakes know this isn't used?
I ran pyflakes, and it uncovered some errors related to symbols that weren't defined, or were defined more than once. These are only minor errors in code that is rarely run, or never run.