Skip to content

Commit

Permalink
ci: run twister with CONFIG_LEGACY_INCLUDE_PATH=n
Browse files Browse the repository at this point in the history
Make sure all tests are run with the legacy include path option
disabled. This should prevent code not using includes with the
<zephyr/...> prefix to pass tests and so getting merged.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
  • Loading branch information
gmarull authored and nashif committed May 9, 2022
1 parent 8f09118 commit e4ebba8
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/clang.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ jobs:
if [ -s testplan.csv ]; then
echo "::set-output name=report_needed::1";
# Full twister but with options based on changes
./scripts/twister --inline-logs -M -N -v --load-tests testplan.csv --retry-failed 2
./scripts/twister --inline-logs -M -N -v --load-tests testplan.csv --retry-failed 2 -x "CONFIG_LEGACY_INCLUDE_PATH=n"
else
# if nothing is run, skip reporting step
echo "::set-output name=report_needed::0";
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/codecov.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ jobs:
export ZEPHYR_BASE=${PWD}
export ZEPHYR_TOOLCHAIN_VARIANT=zephyr
mkdir -p coverage/reports
./scripts/twister -N -v --filter runnable -p ${{ matrix.platform }} --coverage -T tests
./scripts/twister -N -v --filter runnable -p ${{ matrix.platform }} --coverage -T tests -x "CONFIG_LEGACY_INCLUDE_PATH=n"
- name: Generate Coverage Report
run: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/twister.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ jobs:
env:
ZEPHYR_SDK_INSTALL_DIR: /opt/toolchains/zephyr-sdk-0.14.1
CLANG_ROOT_DIR: /usr/lib/llvm-12
TWISTER_COMMON: ' --inline-logs -v -N -M --retry-failed 3 '
TWISTER_COMMON: ' --inline-logs -v -N -M --retry-failed 3 -x "CONFIG_LEGACY_INCLUDE_PATH=n" '
DAILY_OPTIONS: ' -M --build-only --all --no-skipped-report'
PR_OPTIONS: ' --clobber-output --integration --no-skipped-report'
PUSH_OPTIONS: ' --clobber-output -M --no-skipped-report'
Expand Down

0 comments on commit e4ebba8

Please sign in to comment.