Skip to content

Commit

Permalink
[INFRA] New attempt to silence LGTM false positive
Browse files Browse the repository at this point in the history
Previous commit bb6065a from bids-standard#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.
  • Loading branch information
DimitriPapadopoulos committed Aug 27, 2021
1 parent 8345d50 commit 044e673
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 3 additions & 0 deletions lgtm.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
# https://github.com/github/codeql/issues/6517
queries:
- exclude: py/unused-import
3 changes: 2 additions & 1 deletion pdf_build_src/process_markdowns.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@
import numpy as np

sys.path.append("../tools/")
from schemacode import macros # noqa (used in "eval" call later on)
# functions from module macros are called by eval() later on
from schemacode import macros # noqa: F401


def run_shell_cmd(command):
Expand Down

0 comments on commit 044e673

Please sign in to comment.