Fix flake8 F821 errors

This is for using a variable that isn't defined. This only had some false-positives, which are worked around in this commit. But it seems like a good check to be running for the future.
CleverRaven · Sep 16, 2020 · f9e5bb1 · f9e5bb1
1 parent 0fc6ebb
commit f9e5bb1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.flake8 b/.flake8
@@ -22,7 +22,6 @@ ignore =
     E713,
     E722,
     E731,
-    F821,
     F841,
     W291,
     W391,

diff --git a/build-data/osx/dmgsettings.py b/build-data/osx/dmgsettings.py
@@ -4,6 +4,10 @@
 import biplist
 import os.path
 
+# defines is provided by dmgbuild, but pretend to define it here for the
+# beneit of linting tools.
+defines = defines # noqa: F821
+
 # .. Useful stuff ..............................................................
 
 application = defines.get('app', 'Cataclysm.app')
-Original file line number
+Diff line change
@@ Expand Up / @@ -22,7 +22,6 @@ ignore = @@
         E713,
         E722,
         E731,
-        F821,
         F841,
         W291,
         W391,
@@ Expand Down @@