From 3d3c7501612c78e6f2d60db5f0e074b5630f7360 Mon Sep 17 00:00:00 2001 From: Thomas Bui <43018778+Thomas-Boi@users.noreply.github.com> Date: Mon, 14 Mar 2022 10:10:51 -0700 Subject: [PATCH] Fix check icon not running for update icon pr (#1060) --- .github/scripts/icomoon_build.py | 4 ++-- .github/workflows/check_icon_pr.yml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/scripts/icomoon_build.py b/.github/scripts/icomoon_build.py index ab45b33c..56f6544a 100644 --- a/.github/scripts/icomoon_build.py +++ b/.github/scripts/icomoon_build.py @@ -56,7 +56,7 @@ def main(): print("Task completed.") except TimeoutException as e: - util.exit_with_err("Selenium Time Out Error: \n" + str(e)) + util.exit_with_err(Exception("Selenium Time Out Error: \n" + str(e))) except Exception as e: util.exit_with_err(e) finally: @@ -95,7 +95,7 @@ def get_icons_for_building(icomoon_json_path: str, devicon_json_path: str, token return new_icons -def optimize_svgs(new_icons: List[str], icons_folder_path: str): +def optimize_svgs(new_icons: List[dict], icons_folder_path: str): """ Optimize the newly added svgs. This is done in batches since the command line has a limit on characters allowed. diff --git a/.github/workflows/check_icon_pr.yml b/.github/workflows/check_icon_pr.yml index 1247ed8a..6a57bab4 100644 --- a/.github/workflows/check_icon_pr.yml +++ b/.github/workflows/check_icon_pr.yml @@ -4,7 +4,7 @@ jobs: check: name: Check the `devicon.json` and the SVGs' quality runs-on: ubuntu-18.04 - if: startsWith(github.event.pull_request.title, 'new icon') # only checks icon PR + if: startsWith(github.event.pull_request.title, 'new icon') || startsWith(github.event.pull_request.title, 'update icon') # only checks icon PR steps: - uses: actions/checkout@v2