Skip to content

Commit 9b00775

Browse files
committed
actions: manifest: Align with upstream Zephyr
The manifest action now has many more features at version 1.6.0. Update to the latest and start using a tree checkout to diff the manifest, which is compatible with split manifests. This is a second attempt after bf6df10 had to be reverted in f4744e2 due to the fact that the actual action did not support selecting which projects to import. This has now been fixed in: zephyrproject-rtos/action-manifest@1807b4c and included in v1.6.0 of the action. Signed-off-by: Carles Cufi <[email protected]>
1 parent 6147ddf commit 9b00775

File tree

1 file changed

+19
-12
lines changed

1 file changed

+19
-12
lines changed

.github/workflows/manifest.yml

+19-12
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,12 @@
11
name: Manifest
2-
3-
on:
4-
pull_request_target:
2+
on: pull_request_target
53

64
permissions:
75
contents: read
86
pull-requests: write
97

108
jobs:
11-
contribs:
9+
manifest:
1210
runs-on: ubuntu-latest
1311
name: Manifest
1412
steps:
@@ -20,19 +18,28 @@ jobs:
2018
fetch-depth: 0
2119
persist-credentials: false
2220

21+
- name: west setup
22+
env:
23+
BASE_REF: ${{ github.base_ref }}
24+
working-directory: ncs/nrf
25+
run: |
26+
pip3 install west
27+
git config --global user.email "[email protected]"
28+
git config --global user.name "Your Name"
29+
west init -l . || true
30+
# We only import the zephyr manifest
31+
west update zephyr
32+
2333
- name: Manifest
24-
uses: zephyrproject-rtos/action-manifest@16c4cfa380ae2b6fa3daddb1a35032e69422a20f
34+
uses: zephyrproject-rtos/action-manifest@v1.6.0
2535
with:
2636
github-token: ${{ secrets.NCS_GITHUB_TOKEN }}
2737
manifest-path: 'west.yml'
2838
checkout-path: 'ncs/nrf'
39+
use-tree-checkout: 'true'
40+
west-import-flag: 'self'
41+
check-impostor-commits: 'true'
2942
label-prefix: 'manifest-'
3043
verbosity-level: '1'
31-
32-
# Add one label per line. 'manifest' always adds the label 'manifest'.
33-
# 'CI-all-test:zephyr;nrfxlib,' adds the 'CI-all-test' label when the
34-
# zephyr module or the nrfxlib module is changed. Each line is comma-
35-
# separated.
36-
labels: >
37-
manifest
44+
labels: 'manifest'
3845
dnm-labels: 'DNM'

0 commit comments

Comments
 (0)