Skip to content

Commit 110128d

Browse files
workflows: make compliance fast, real fast
Use ubuntu image for compliance (3 minuts faster). Slim west update (1 minute faster). Total compliance stage around 40 seconds now. Signed-off-by: Giacomo Dematteis <giacomo.dematteis@nordicsemi.no>
1 parent 05fca9e commit 110128d

File tree

2 files changed

+13
-6
lines changed

2 files changed

+13
-6
lines changed

.github/workflows/build.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ jobs:
9292
echo "MEMFAULT_SW_TYPE=${{ inputs.memfault_sw_type }}" >> $GITHUB_ENV
9393
fi
9494
95-
# Out-of-box firmware build
95+
# Asset Tracker Template firmware build
9696

9797
- name: Build nrf91 firmware
9898
working-directory: asset-tracker-template/app

.github/workflows/compliance.yml

+12-5
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,15 @@ jobs:
1010
compliance_job:
1111
runs-on: ubuntu-24.04
1212
name: Run compliance checks on patch series (PR)
13-
container: ghcr.io/zephyrproject-rtos/ci:v0.27.4
1413

1514
# Skip job if it was triggered by Renovate Bot
1615
if: ${{ !contains(github.actor, 'renovate') }}
1716

1817
steps:
18+
- name: Pip install
19+
run: |
20+
pip install unidiff yamllint junitparser python-magic west lxml gitlint
21+
1922
- name: Checkout the code
2023
uses: actions/checkout@v4
2124
with:
@@ -27,10 +30,15 @@ jobs:
2730
working-directory: asset-tracker-template
2831
run: |
2932
west init -l .
30-
west config manifest.group-filter +bsec
31-
west config build.sysbuild True
33+
west config manifest.project-filter -- \
34+
"-bsim,-wfa-qt-control-app,-mcuboot,-qcbor,-mbedtls,-oberon-psa-crypto,\
35+
-nrfxlib,-trusted-firmware-m,-psa-arch-tests,-matter,-cjson,-azure-sdk-for-c,\
36+
-cirrus,-openthread,-suit-generator,-suit-processor,-cmock,-memfault-firmware-sdk,\
37+
-canopennode,-chre,-lz4,-nanopb,-tf-m-tests,-zscilib,-cmsis-dsp,-cmsis-nn,-hostap,\
38+
-liblc3,-loramac-node,-lvgl,-nrf_hw_models,-nrf_wifi,-open-amp,-picolibc,-uoscore-uedhoc,\
39+
-zcbor,-soc-hwmv1,-coremark,-cmsis,-edtt,-fatfs,-hal_nordic,-hal_st,-hal_wurthelektronik,\
40+
-libmetal,-littlefs,-mipi-sys-t,-net-tools,-segger,-tinycrypt"
3241
west update -o=--depth=1 -n
33-
west blobs fetch hal_nordic
3442
3543
- name: Run Compliance Tests
3644
id: compliance
@@ -48,7 +56,6 @@ jobs:
4856
-m Nits \
4957
-m pylint \
5058
-m checkpatch \
51-
-m KconfigBasic \
5259
-c origin/${BASE_REF}.. || \
5360
echo "COMPLIANCE_FAILED=true" >> $GITHUB_ENV
5461

0 commit comments

Comments
 (0)