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 #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 Sep 1, 2021
1 parent f21e9b2 commit c13e290
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 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):
Expand Down

0 comments on commit c13e290

Please sign in to comment.