[INFRA] New attempt to silence LGTM false positive

DimitriPapadopoulos · DimitriPapadopoulos · commit 68890a4bb034 · 2021-09-13T14:23:05.000+02:00
Previous commit bb6065a from #853 does not work around the LGTM alert. We attempt to silence the LGTM alert using an lgtm.yml file, and use standard Flake8 noqa suppression comments to document the issue at hand.
diff --git a/lgtm.yml b/lgtm.yml
@@ -0,0 +1,3 @@
+queries:
+  # https://github.com/github/codeql/issues/6517
+  - exclude: py/unused-import
diff --git a/pdf_build_src/process_markdowns.py b/pdf_build_src/process_markdowns.py
@@ -16,7 +16,8 @@
 import numpy as np
 
 sys.path.append("../tools/")
-from mkdocs_macros_bids import macros  # noqa   (used in "eval" call later on)
+# functions from module macros are called by eval() later on
+from mkdocs_macros_bids import macros  # noqa: F401
 
 
 def run_shell_cmd(command):

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+queries:`
	`2`	`+ # https://github.com/github/codeql/issues/6517`
	`3`	`+ - exclude: py/unused-import`