ci: adding testspec for test filtering #516
Conversation
|
You can find the documentation preview for this PR here. |
.github/test-spec.yml
Outdated
| - "sysbuild/**/*" | ||
| - "tests/subsys/kmu/**/*" | ||
| - "west.yml" | ||
| - any: |
There was a problem hiding this comment.
any block should be removed. If I remember correctly how things work, that will cause CI-dfu-test and CI-bm-samples run on every change not matching patterns in any block
.github/test-spec.yml
Outdated
| - "Jenkinsfile" | ||
| - "Kconfig.nrf_bm" | ||
| - "west.yml" | ||
| - any: |
There was a problem hiding this comment.
any block should be removed. If I remember correctly how things work, that will cause CI-dfu-test and CI-bm-samples run on every change not matching patterns in any block
There was a problem hiding this comment.
doesnt that mean I have to remove everything other than any block ?
.github/test-spec.yml
Outdated
| - "include/**/*" | ||
| - "lib/**/*" | ||
| - "modules/**/*" | ||
| - "samples/**/*" |
There was a problem hiding this comment.
| - "samples/**/*" | |
| - "samples/boot/**/*" |
.github/test-spec.yml
Outdated
| - "sysbuild/**/*" | ||
| - "tests/subsys/kmu/**/*" | ||
| - "west.yml" | ||
| - any: |
There was a problem hiding this comment.
As far as I understand this any block is not needed since preceding matches will exclude below entries.
|
|
||
| "CI-bm-samples": | ||
| - "applications/**/*" | ||
| - "components/softdevice/**/*" |
There was a problem hiding this comment.
Should not be needed to update this if we add more to components.
| - "components/softdevice/**/*" | |
| - "components/**/*" |
Signed-off-by: Nirmal Krishna <[email protected]>
4 participants
No description provided.