Skip to content

Commit

Permalink
Fix CI for Bitcoin Nanos being a separate branch
Browse files Browse the repository at this point in the history
  • Loading branch information
fbeutin-ledger committed Jun 26, 2024
1 parent dcf9c6f commit 4c57f98
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions .github/workflows/reusable_swap_functional_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,10 @@ on:
required: false
default: 'develop'
type: string
branch_for_bitcoin_nanos:
required: false
default: 'nanos'
type: string
repo_for_bitcoin:
required: false
default: 'LedgerHQ/app-bitcoin-new'
Expand Down Expand Up @@ -125,6 +129,9 @@ jobs:
- name: bitcoin
repo: ${{ inputs.repo_for_bitcoin }}
branch: ${{ inputs.branch_for_bitcoin }}
- name: bitcoin
repo: ${{ inputs.repo_for_bitcoin }}
branch: ${{ inputs.branch_for_bitcoin_nanos }}
- name: solana
repo: ${{ inputs.repo_for_solana }}
branch: ${{ inputs.branch_for_solana }}
Expand All @@ -141,12 +148,14 @@ jobs:
uses: LedgerHQ/ledger-app-workflows/.github/workflows/reusable_build.yml@v1
with:
app_repository: ${{ matrix.coin.repo }}
app_branch_name: ${{ matrix.coin.branch }}
app_branch_name: ${{ matrix.coin.branch }}
# Set the coin variant to build with COIN or CHAIN depending on the app + flags for ETH compilation
# As a rule of thumb we don't care about giving unexisting flags to other apps, it has no effect
flags: "COIN=${{ matrix.coin.name }} CHAIN=${{ matrix.coin.name }} CAL_TEST_KEY=1 DOMAIN_NAME_TEST_KEY=1 SET_PLUGIN_TEST_KEY=1 NFT_TEST_KEY=1"
upload_app_binaries_artifact: libraries_binaries-${{ matrix.coin.name }}
upload_app_binaries_artifact: libraries_binaries-${{ matrix.coin.name }}-${{ matrix.coin.branch }}
upload_as_lib_artifact: ${{ matrix.coin.name }}
# Bitcoin has two branches depending on the device, use ternary expression to run nanos or nanox
run_for_devices: ${{ matrix.coin.name != 'bitcoin' && '["nanos", "nanox", "nanosp", "stax"]' || matrix.coin.branch == inputs.branch_for_bitcoin_nanos && '["nanos"]' || '["nanox", "nanosp", "stax"]' }}

merge_libraries_build:
name: Merge built libraries
Expand Down

0 comments on commit 4c57f98

Please sign in to comment.