Skip to content

Commit ba97e9d

Browse files
nordic-piksrlubos
authored andcommitted
[nrf fromlist] 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. Upstream PR: zephyrproject-rtos/zephyr#75927 Signed-off-by: Piotr Kosycarz <[email protected]>
1 parent 2af3400 commit ba97e9d

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)