Skip to content
This repository was archived by the owner on Apr 6, 2022. It is now read-only.

Commit 921d4f3

Browse files
ulfalizernashif
authored andcommitted
check_compliance.py: Fix Nits test after prepare() was removed
Needs to be updated after commit 187a2fc ("check_compliance.py: Remove prepare(), add 'path' class variable, misc.") was merged. Signed-off-by: Ulf Magnusson <[email protected]>
1 parent 61d75fd commit 921d4f3

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

scripts/check_compliance.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -664,15 +664,14 @@ class Nits(ComplianceTest):
664664
Checks various nits in added/modified files. Doesn't check stuff that's
665665
already covered by e.g. checkpatch.pl and pylint.
666666
"""
667-
_name = "Nits"
668-
_doc = "https://docs.zephyrproject.org/latest/contribute/#coding-style"
667+
name = "Nits"
668+
doc = "https://docs.zephyrproject.org/latest/contribute/#coding-style"
669+
path_hint = GIT_TOP
669670

670671
def run(self):
671-
self.prepare(GIT_TOP)
672-
673672
# Loop through added/modified files
674673
for fname in git("diff", "--name-only", "--diff-filter=d",
675-
self.commit_range).splitlines():
674+
COMMIT_RANGE).splitlines():
676675
is_kconfig = "Kconfig" in fname
677676

678677
if is_kconfig:

0 commit comments

Comments
 (0)