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 13, 2021
1 parent 2050b32 commit 49fbdac
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
4 changes: 4 additions & 0 deletions .lgtm.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
---
queries:
# https://lgtm.com/rules/6770079/
- 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 49fbdac

Please sign in to comment.