Skip to content

Commit

Permalink
ci: Build test_app only on relevant changes
Browse files Browse the repository at this point in the history
  • Loading branch information
tore-espressif committed Mar 24, 2023
1 parent c8227c0 commit 67d2785
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 9 deletions.
12 changes: 4 additions & 8 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -1,12 +1,8 @@
# Checklist for new Board Support package or Component
# ESP-BSP Pull Request checklist

- [ ] Component contains License
- [ ] Component contains README.md
- [ ] Project [README.md](../README.md) updated
- [ ] Component contains idf_component.yml file with `url` field defined
- [ ] Component was added to CI [upload job](https://github.com/espressif/esp-bsp/blob/master/.github/workflows/upload_component.yml#L17)
- [ ] New files were added to CI build job
- [ ] _Optional:_ Component contains unit tests
> Note: For new BSPs create a PR with this [link](https://github.com/espressif/esp-bsp/compare/main...my-branch?quick_pull=1&template=pr_template_bsp.md).
- [ ] Version of modified component bumped
- [ ] CI passing

# Change description
Expand Down
14 changes: 14 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE/pr_template_bsp.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
# Checklist for new Board Support package or Component

- [ ] Component contains License
- [ ] Component contains README.md
- [ ] Project [README.md](../README.md) updated
- [ ] Component contains idf_component.yml file with `url` field defined
- [ ] Component was added to CI [upload job](https://github.com/espressif/esp-bsp/blob/master/.github/workflows/upload_component.yml#L17)
- [ ] New files were added to CI build job
- [ ] New BSP definitions added to [build_example_for_all_bsps.sh](./build_example_for_all_bsps.sh) and to [bsp_ext.py](../examples/bsp_ext.py)
- [ ] _Optional:_ Component contains unit tests
- [ ] CI passing

# Change description
_Please describe your change here_
6 changes: 5 additions & 1 deletion .github/workflows/build_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@ name: Build Test Application
on:
pull_request:
types: [opened, reopened, synchronize]
paths:
- 'components/**'
- 'test_app/**'
- '.github/workflows/build_test.yml'

jobs:
build:
Expand All @@ -20,9 +24,9 @@ jobs:
- name: Build ESP-BSP Test Application
env:
IDF_TARGET: ${{ matrix.idf_target }}
working-directory: test_app
shell: bash
run: |
cd test_app
. ${IDF_PATH}/export.sh
export PEDANTIC_FLAGS="-DIDF_CI_BUILD -Werror -Werror=deprecated-declarations -Werror=unused-variable -Werror=unused-but-set-variable -Werror=unused-function"
export EXTRA_CFLAGS="${PEDANTIC_FLAGS} -Wstrict-prototypes"
Expand Down

0 comments on commit 67d2785

Please sign in to comment.