Allows to run icon checks based on matrix (quicker)

Author: cedelavergne-ledger

.github/workflows/_check_icons.yml

@@ -21,10 +21,18 @@ on:
         description: 'The name of the artifact containing the built manifest'
         required: true
         type: string
+      run_for_devices:
+        description: 'The list of device(s) on which the checking will run'
+        required: true
+        type: string
 
 jobs:
   check_icons:
     name: Check icons
+    strategy:
+      fail-fast: false
+      matrix:
+        device: ${{ fromJSON(inputs.run_for_devices) }}
     runs-on: ubuntu-22.04
 
     steps:
@@ -50,5 +58,5 @@ jobs:
 
       - name: Run script
         run: |
-            ./ledger-app-workflows/scripts/check_all.sh -c icons \
+            ./ledger-app-workflows/scripts/check_all.sh -c icons -t ${{ matrix.device }} \
               -a ./app-repository -m ${{ inputs.download_manifest_artifact_name }}

.github/workflows/_check_makefile.yml

@@ -21,10 +21,18 @@ on:
         description: 'The name of the artifact containing the built manifest'
         required: true
         type: string
+      run_for_devices:
+        description: 'The list of device(s) on which the checking will run'
+        required: true
+        type: string
 
 jobs:
   check_makefile:
     name: Check Makefile
+    strategy:
+      fail-fast: false
+      matrix:
+        device: ${{ fromJSON(inputs.run_for_devices) }}
     runs-on: ubuntu-22.04
 
     steps:
@@ -50,5 +58,5 @@ jobs:
 
       - name: Run script
         run: |
-            ./ledger-app-workflows/scripts/check_all.sh -c makefile \
+            ./ledger-app-workflows/scripts/check_all.sh -c makefile -t ${{ matrix.device }} \
               -a ./app-repository -m ${{ inputs.download_manifest_artifact_name }}

.github/workflows/reusable_guidelines_enforcer.yml

@@ -60,12 +60,13 @@ jobs:
 
   call_check_icons:
     name: Dispatch check
-    needs: [call_get_workflow_version, call_get_app_manifest]
+    needs: [call_get_app_metadata, call_get_workflow_version, call_get_app_manifest]
     uses: ./.github/workflows/_check_icons.yml
     with:
       app-repository: ${{ inputs.app_repository }}
       download_manifest_artifact_name: manifests
       ledger-app-workflows_ref: ${{ needs.call_get_workflow_version.outputs.version }}
+      run_for_devices: ${{ needs.call_get_app_metadata.outputs.compatible_devices }}
 
   call_check_app_load_params:
     name: Dispatch check
@@ -77,12 +78,13 @@ jobs:
 
   call_check_makefile:
     name: Dispatch check
-    needs: [call_get_workflow_version, call_get_app_manifest]
+    needs: [call_get_app_metadata, call_get_workflow_version, call_get_app_manifest]
     uses: ./.github/workflows/_check_makefile.yml
     with:
       app-repository: ${{ inputs.app_repository }}
       download_manifest_artifact_name: manifests
       ledger-app-workflows_ref: ${{ needs.call_get_workflow_version.outputs.version }}
+      run_for_devices: ${{ needs.call_get_app_metadata.outputs.compatible_devices }}
 
   call_check_readme:
     name: Dispatch check

scripts/check_all.sh

@@ -198,13 +198,13 @@ call_step() {
             fi
             ;;
         "icons")
-            COMMAND="${dirName}/check_icons.sh ${APP_DIR} ${REPO_NAME} ${MANIFEST_DIR}"
+            COMMAND="${dirName}/check_icons.sh ${APP_DIR} ${REPO_NAME} ${MANIFEST_DIR} ${TARGET}"
             ;;
         "app_load_params")
             COMMAND="python3 ${DATABASE_DIR}/scripts/app_load_params_check.py --database_path ${DATABASE_DIR}/app-load-params-db.json --app_manifests_path ${MANIFEST_DIR}"
             ;;
         "makefile")
-            COMMAND="${dirName}/check_makefile.sh ${APP_DIR} ${REPO_NAME} ${MANIFEST_DIR}"
+            COMMAND="${dirName}/check_makefile.sh ${APP_DIR} ${REPO_NAME} ${MANIFEST_DIR} ${TARGET}"
             ;;
         "readme")
             COMMAND="${dirName}/check_readme.sh ${APP_DIR} ${REPO_NAME}"

scripts/check_icons.sh

@@ -91,6 +91,7 @@ check_glyph() (
         colors_nb=$(echo "$content" | grep "Colors: " | rev | cut -d' ' -f1 | rev)
         if [[ "$colors_nb" -gt 16 ]]; then
             log_error "4bpp glyphs can't have more than 16 colors, $colors_nb found"
+            error=1
         fi
 
         # Be somewhat tolerant to different possible wordings for depth "8 bit" "8-bit" "8/8 bit" etc
@@ -155,12 +156,17 @@ main() (
     repo="$1"
     repo_name="$2"
     manifests_dir="$3"
+    target="$4"
 
     all_glyph_files=""
     declare -A icons_and_devices
 
     # Parse all manifest files
-    manifests_list=$(find "$manifests_dir" -type f -name "*.json")
+    if [[ -n "${target}" ]]; then
+        manifests_list="${manifests_dir}/manifest_${target}.json"
+    else
+        manifests_list=$(find "$manifests_dir" -type f -name "*.json")
+    fi
     while IFS= read -r manifest; do
         log_info "Checking manifest $manifest"
 

scripts/check_makefile.sh

@@ -10,6 +10,7 @@ main() (
     repo="$1"
     repo_name="$2"
     manifests_dir="$3"
+    target="$4"
 
     declare -A variants_array
     declare -A appnames_array
@@ -22,7 +23,11 @@ main() (
     fi
 
     # Parse all manifest files
-    manifests_list=$(find "$manifests_dir" -type f -name "*.json")
+    if [[ -n "${target}" ]]; then
+        manifests_list="${manifests_dir}/manifest_${target}.json"
+    else
+        manifests_list=$(find "$manifests_dir" -type f -name "*.json")
+    fi
     while IFS= read -r manifest; do
         log_info "Checking manifest $manifest"