Merge pull request #548 from LedgerHQ/release/1.10.4

App release 1.10.4

Author: apaillier-ledger

.github/workflows/auto-author-assign.yml

@@ -1,3 +1,4 @@
+---
 name: 'Auto Author Assign'
 
 on:

.github/workflows/build-workflow.yml

@@ -1,39 +1,19 @@
+---
 name: Tests
 
 on:
+  workflow_dispatch:
   push:
-    branches:
-      - master
-  pull_request:
     branches:
       - master
       - develop
-  workflow_dispatch:
+  pull_request:
 
 jobs:
-  scan-build:
-    name: Clang Static Analyzer
-    runs-on: ubuntu-latest
-    container:
-      image: ghcr.io/ledgerhq/ledger-app-builder/ledger-app-builder-lite:latest
 
-    steps:
-      - uses: actions/checkout@v3
-
-      - name: Build with Clang Static Analyzer
-        run: |
-          make clean
-          scan-build --use-cc=clang -analyze-headers -enable-checker security -enable-checker unix -enable-checker valist -o scan-build --status-bugs make default
-
-      - uses: actions/upload-artifact@v3
-        if: failure()
-        with:
-          name: scan-build
-          path: scan-build
-
-# =====================================================
-#            ZEMU TESTS
-# =====================================================
+  # =====================================================
+  #            ZEMU TESTS
+  # =====================================================
 
   building_for_e2e_zemu_tests:
     name: Building binaries for E2E Zemu tests
@@ -97,9 +77,9 @@ jobs:
       - name: Run zemu tests
         run: cd tests/zemu/ && yarn test
 
-# =====================================================
-#            SPECULOS TESTS
-# =====================================================
+  # =====================================================
+  #            SPECULOS TESTS
+  # =====================================================
 
 
   building_for_e2e_speculos_tests:
@@ -114,9 +94,9 @@ jobs:
       - name: Build testing binaries
         run: |
           mkdir tests/speculos/elfs
-          make clean && make -j DEBUG=1 NFT_TESTING_KEY=1 BOLOS_SDK=$NANOS_SDK && mv bin/app.elf tests/speculos/elfs/nanos.elf
-          make clean && make -j DEBUG=1 NFT_TESTING_KEY=1 BOLOS_SDK=$NANOX_SDK && mv bin/app.elf tests/speculos/elfs/nanox.elf
-          make clean && make -j DEBUG=1 NFT_TESTING_KEY=1 BOLOS_SDK=$NANOSP_SDK && mv bin/app.elf tests/speculos/elfs/nanosp.elf
+          make clean && make -j DEBUG=1 NFT_STAGING_KEY=1 BOLOS_SDK=$NANOS_SDK && mv bin/app.elf tests/speculos/elfs/nanos.elf
+          make clean && make -j DEBUG=1 NFT_STAGING_KEY=1 BOLOS_SDK=$NANOX_SDK && mv bin/app.elf tests/speculos/elfs/nanox.elf
+          make clean && make -j DEBUG=1 NFT_STAGING_KEY=1 BOLOS_SDK=$NANOSP_SDK && mv bin/app.elf tests/speculos/elfs/nanosp.elf
 
       - name: Upload app binaries
         uses: actions/upload-artifact@v3
@@ -162,21 +142,60 @@ jobs:
           pytest --model ${{ matrix.model }} --path ./elfs/${{ matrix.model }}.elf --display headless
 
 
-# =====================================================
-#            RAGGER TESTS
-# =====================================================
+  # =====================================================
+  #            RAGGER TESTS
+  # =====================================================
 
   build_ragger_elfs:
     name: Build app for Ragger tests
     uses: LedgerHQ/ledger-app-workflows/.github/workflows/reusable_build.yml@v1
     with:
       upload_app_binaries_artifact: "ragger_elfs"
-      flags: "DEBUG=1 CAL_CI_KEY=1 DOMAIN_NAME_TEST_KEY=1"
+      flags: "DEBUG=1 CAL_TEST_KEY=1 DOMAIN_NAME_TEST_KEY=1 SET_PLUGIN_TEST_KEY=1 NFT_TEST_KEY=1"
 
   jobs-ragger-tests:
     name: Run Ragger tests
     needs: build_ragger_elfs
     uses: LedgerHQ/ledger-app-workflows/.github/workflows/reusable_ragger_tests.yml@v1
     with:
       download_app_binaries_artifact: "ragger_elfs"
-      test_dir: tests/ragger
+
+  # =====================================================
+  #            STATIC ANALYSIS
+  # =====================================================
+
+  # Static analysis on the main ETH chain is covered by the guidelines enforcer
+  scan-build:
+    name: Clang Static Analyzer on altcoin
+    runs-on: ubuntu-latest
+    container:
+      image: ghcr.io/ledgerhq/ledger-app-builder/ledger-app-builder:latest
+    strategy:
+      fail-fast: false
+      matrix:
+        device: ["nanos", "nanos2", "nanox", "stax"]
+
+    steps:
+      - name: Clone
+        uses: actions/checkout@v3
+        with:
+          submodules: recursive
+
+      - name: Build with Clang Static Analyzer
+        run: |
+          eval "BOLOS_SDK=\$$(echo ${{ matrix.device }} | tr [:lower:] [:upper:])_SDK" && \
+          echo "BOLOS_SDK value will be: ${BOLOS_SDK}" && \
+          make -j ENABLE_SDK_WERROR=1 BOLOS_SDK=${BOLOS_SDK} CHAIN=polygon scan-build
+
+      - uses: actions/upload-artifact@v3
+        if: failure()
+        with:
+          name: scan-build
+          path: scan-build
+
+      - name: Upload scan result
+        if: failure()
+        uses: actions/upload-artifact@v3
+        with:
+          name: scan-build
+          path: scan-build

.github/workflows/ci-workflow.yml

@@ -1,26 +1,26 @@
+---
 name: Code style check
 
+# This workflow will run linting checks to ensure a level of uniformization among all Ledger applications.
+#
+# The presence of this workflow is mandatory as a minimal level of linting is required.
+# You are however free to modify the content of the .clang-format file and thus the coding style of your application.
+# We simply ask you to not diverge too much from the linting of the Boilerplate application.
+
 on:
+  workflow_dispatch:
   push:
     branches:
-    - master
+      - master
+      - main
+      - develop
   pull_request:
-    branches:
-    - master
-    - develop
 
 jobs:
-  job_lint:
-    name: Lint
-    runs-on: ubuntu-latest
-
-    steps:
-      - name: Clone
-        uses: actions/checkout@v3
-
-      - name: Lint
-        uses: DoozyX/[email protected]
-        with:
-          source: "./"
-          extensions: "h,c"
-          clangFormatVersion: 12.0.1
+  check_linting:
+    name: Check linting using the reusable workflow
+    uses: LedgerHQ/ledger-app-workflows/.github/workflows/reusable_lint.yml@v1
+    with:
+      source: './'
+      extensions: 'h,c'
+      version: 12