From 4dce9d0d3711c74cff05c15f1f8eb2bb2e00da5d Mon Sep 17 00:00:00 2001 From: Ankush Menat Date: Mon, 29 Jan 2024 16:47:04 +0530 Subject: [PATCH] ci: skip UI test for PO files (#24581) --- .github/helper/roulette.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/helper/roulette.py b/.github/helper/roulette.py index e3b212fa89..ebca901c95 100644 --- a/.github/helper/roulette.py +++ b/.github/helper/roulette.py @@ -73,8 +73,9 @@ def has_label(pr_number, label, repo="frappe/frappe"): ) -def is_py(file): - return file.endswith("py") +def is_server_side_code(file): + """File exclusively affects server side code""" + return file.endswith("py") or file.endswith(".po") def is_ci(file): @@ -112,7 +113,7 @@ if __name__ == "__main__": ci_files_changed = any(f for f in files_list if is_ci(f)) only_docs_changed = len(list(filter(is_docs, files_list))) == len(files_list) only_frontend_code_changed = len(list(filter(is_frontend_code, files_list))) == len(files_list) - updated_py_file_count = len(list(filter(is_py, files_list))) + updated_py_file_count = len(list(filter(is_server_side_code, files_list))) only_py_changed = updated_py_file_count == len(files_list) if has_skip_ci_label(pr_number, repo):