Skip to content

Commit

Permalink
[INFRA] Add .lgtm.yml file for better usage of LGTM CI tool
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 authored Sep 13, 2021
1 parent 2050b32 commit 811e17e
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
8 changes: 8 additions & 0 deletions .lgtm.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
---
# This file contains configuration for the LGTM tool: https://lgtm.com/
# The bids-specification repository is continuously scanned by the LGTM tool
# for any security and/or code vulnerabilities. You can find the alert here:
# https://lgtm.com/projects/g/bids-standard/bids-specification/
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 811e17e

Please sign in to comment.