-
Notifications
You must be signed in to change notification settings - Fork 4
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add first version of app-params database and checker
- Loading branch information
Xavier Chapron
committed
Apr 13, 2023
1 parent
9302b19
commit bf16d97
Showing
7 changed files
with
1,718 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
name: Check APP_LOAD_PARAMS | ||
|
||
on: | ||
workflow_call: | ||
inputs: | ||
ledger-app-workflows_ref: | ||
description: 'The current reference in use for the ledger-app-workflow repository' | ||
required: true | ||
type: string | ||
download_manifest_artifact_name: | ||
description: 'The name of the artifact containing the built manifest' | ||
required: true | ||
type: string | ||
|
||
jobs: | ||
check_app_paramss: | ||
name: Check APP_LOAD_PARAMS | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Clone workflows repository | ||
uses: actions/checkout@v3 | ||
with: | ||
repository: LedgerHQ/ledger-app-workflows | ||
path: ./ledger-app-workflows | ||
ref: ${{ inputs.ledger-app-workflows_ref }} | ||
|
||
- name: Download manifest | ||
uses: actions/download-artifact@v3 | ||
with: | ||
name: ${{ inputs.download_manifest_artifact_name }} | ||
path: ${{ inputs.download_manifest_artifact_name }} | ||
|
||
- name: Download app params database | ||
run: curl https://raw.githubusercontent.com/LedgerHQ/ledger-app-workflows/xch/app-params-check/app-params-database.json > app-params-database.json | ||
|
||
- name: Run script | ||
run: | | ||
python3 ./ledger-app-workflows/scripts/makefile_app_params_check.py \ | ||
--database_path app-params-database.json \ | ||
--app_manifests_path ${{ inputs.download_manifest_artifact_name }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
name: Database lint CI | ||
|
||
on: | ||
workflow_dispatch: | ||
push: | ||
branches: | ||
- master | ||
- main | ||
- develop | ||
pull_request: | ||
|
||
jobs: | ||
database_lint: | ||
name: Database lint | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v3 | ||
|
||
- name: Run JQ check | ||
run: jq -e . app-params-database.json >/dev/null | ||
|
||
- name: Run linting check | ||
run: scripts/gen_database.py --database_path app-params-database.json --check_lint |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -30,3 +30,4 @@ jobs: | |
with: | ||
builtin: clear,rare | ||
check_filenames: true | ||
exclude_file: app-params-database.json |
Oops, something went wrong.