Ignore unused globals for now as there are some legacy ones we don't want to touch.

Author: jonasbardino

.github/workflows/python-sanity-check.yml

@@ -36,8 +36,9 @@ jobs:
         # TODO: fix unittest dir and remove from excludes
         # TODO: fix or migrate legacy grsfs-fuse dir and remove from excludes
         # TODO: should tests be included here - currently fail on py3-only FileNotFoundError, etc.
+        # TODO: fix e.g. '`global logger` is unused' and remove F824 from ignores
         # stop the build if there are Python syntax errors or undefined names
-        flake8 . --exclude=tests,state,envhelp,fixture,output,unittest,grsfs-fuse,irclib.py,seafile-seahub_settings-template.py --count --select=E9,F63,F7,F82 --show-source --statistics
+        flake8 . --exclude=tests,state,envhelp,fixture,output,unittest,grsfs-fuse,irclib.py,seafile-seahub_settings-template.py --count --select=E9,F63,F7,F82 --ignore=F824 --show-source --statistics
         # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
         flake8 . --exclude=tests,state,envhelp,fixture,output,unittest,grsfs-fuse,vm-proxy,irclib.py,seafile-seahub_settings-template.py --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
         # NOTE: for interactive clean up one can use current ignores