Skip to content

Commit b8ae3a4

Browse files
nordic-piksnashif
authored andcommitted
scripts: ci: test_plan: use find_modules only when commits are provided
Current implementation will not work if comits were not provided. ie. use case with list of changed files will fail as args.commits is None. Signed-off-by: Piotr Kosycarz <[email protected]>
1 parent 8d5a7ac commit b8ae3a4

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

scripts/ci/test_plan.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ def get_plan(self, options, integration=False, use_testsuite_root=True):
149149
os.remove(fname)
150150

151151
def find_modules(self):
152-
if 'west.yml' in self.modified_files:
152+
if 'west.yml' in self.modified_files and args.commits is not None:
153153
print(f"Manifest file 'west.yml' changed")
154154
print("=========")
155155
old_manifest_content = repo_to_scan.git.show(f"{args.commits[:-2]}:west.yml")

0 commit comments

Comments
 (0)