Commit b8ae3a41 authored by Piotr Kosycarz's avatar Piotr Kosycarz Committed by Anas Nashif
Browse files

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: default avatarPiotr Kosycarz <piotr.kosycarz@nordicsemi.no>
parent 8d5a7ac1

scripts/ci/test_plan.py

+1 −1
Changes for scripts/ci/test_plan.py: 1 added line, 1 removed line.
Original line number Diff line number Diff line
@@ -149,7 +149,7 @@ class Filters: 
            os.remove(fname)



    def find_modules(self):

        if 'west.yml' in self.modified_files:

        if 'west.yml' in self.modified_files and args.commits is not None:

            print(f"Manifest file 'west.yml' changed")

            print("=========")

            old_manifest_content = repo_to_scan.git.show(f"{args.commits[:-2]}:west.yml")

CRA Git | Maintained and supported by SUSTech CRA and CCSE