ci(common): Run all test on push event

Added a description about the release process to the contributing guide
This commit is contained in:
David Cermak
2023-03-16 08:31:18 +01:00
parent c92ce075fd
commit 4c7720a2c1
8 changed files with 42 additions and 16 deletions

View File

@ -9,7 +9,7 @@ on:
jobs: jobs:
build_asio: build_asio:
if: contains(github.event.pull_request.labels.*.name, 'asio') if: contains(github.event.pull_request.labels.*.name, 'asio') || github.event_name == 'push'
name: Build name: Build
strategy: strategy:
matrix: matrix:
@ -58,7 +58,9 @@ jobs:
target_tests_asio: target_tests_asio:
# Skip running on forks since it won't have access to secrets # Skip running on forks since it won't have access to secrets
if: ${{ contains(github.event.pull_request.labels.*.name, 'asio') && (github.repository == 'espressif/esp-protocols') }} if: |
github.repository == 'espressif/esp-protocols' &&
( contains(github.event.pull_request.labels.*.name, 'asio') || github.event_name == 'push' )
name: Target tests name: Target tests
strategy: strategy:
matrix: matrix:

View File

@ -9,7 +9,7 @@ on:
jobs: jobs:
build_mdns: build_mdns:
if: contains(github.event.pull_request.labels.*.name, 'mdns') if: contains(github.event.pull_request.labels.*.name, 'mdns') || github.event_name == 'push'
name: Build name: Build
strategy: strategy:
matrix: matrix:
@ -42,7 +42,7 @@ jobs:
if-no-files-found: error if-no-files-found: error
build_mdns_app: build_mdns_app:
if: contains(github.event.pull_request.labels.*.name, 'mdns') if: contains(github.event.pull_request.labels.*.name, 'mdns') || github.event_name == 'push'
name: Build Test Apps name: Build Test Apps
strategy: strategy:
matrix: matrix:
@ -91,7 +91,9 @@ jobs:
target_test_apps_mdns: target_test_apps_mdns:
# Skip running on forks since it won't have access to secrets # Skip running on forks since it won't have access to secrets
if: ${{ contains(github.event.pull_request.labels.*.name, 'mdns') && (github.repository == 'espressif/esp-protocols') }} if: |
github.repository == 'espressif/esp-protocols' &&
( contains(github.event.pull_request.labels.*.name, 'mdns') || github.event_name == 'push' )
name: Target Test Apps name: Target Test Apps
strategy: strategy:
matrix: matrix:
@ -129,7 +131,9 @@ jobs:
target_tests_mdns: target_tests_mdns:
# Skip running on forks since it won't have access to secrets # Skip running on forks since it won't have access to secrets
if: ${{ contains(github.event.pull_request.labels.*.name, 'mdns') && (github.repository == 'espressif/esp-protocols') }} if: |
github.repository == 'espressif/esp-protocols' &&
( contains(github.event.pull_request.labels.*.name, 'mdns') || github.event_name == 'push' )
name: Target Example and Unit tests name: Target Example and Unit tests
strategy: strategy:
matrix: matrix:

View File

@ -9,7 +9,7 @@ on:
jobs: jobs:
host_test_mdns: host_test_mdns:
if: contains(github.event.pull_request.labels.*.name, 'mdns') if: contains(github.event.pull_request.labels.*.name, 'mdns') || github.event_name == 'push'
name: Host test name: Host test
runs-on: ubuntu-20.04 runs-on: ubuntu-20.04
container: espressif/idf:latest container: espressif/idf:latest
@ -33,7 +33,7 @@ jobs:
cat ip.txt cat ip.txt
build_afl_host_test_mdns: build_afl_host_test_mdns:
if: contains(github.event.pull_request.labels.*.name, 'mdns') if: contains(github.event.pull_request.labels.*.name, 'mdns') || github.event_name == 'push'
name: Build AFL host test name: Build AFL host test
strategy: strategy:
matrix: matrix:

View File

@ -9,7 +9,7 @@ on:
jobs: jobs:
build_esp_modem: build_esp_modem:
if: contains(github.event.pull_request.labels.*.name, 'modem') if: contains(github.event.pull_request.labels.*.name, 'modem') || github.event_name == 'push'
name: Build name: Build
strategy: strategy:
matrix: matrix:
@ -47,7 +47,7 @@ jobs:
python ./ci/build_apps.py components/esp_modem/examples/${{ matrix.example }} python ./ci/build_apps.py components/esp_modem/examples/${{ matrix.example }}
host_test_esp_modem: host_test_esp_modem:
if: contains(github.event.pull_request.labels.*.name, 'modem') if: contains(github.event.pull_request.labels.*.name, 'modem') || github.event_name == 'push'
name: Host Tests name: Host Tests
runs-on: ubuntu-20.04 runs-on: ubuntu-20.04
container: espressif/idf:release-v4.3 container: espressif/idf:release-v4.3
@ -84,7 +84,7 @@ jobs:
files: esp-protocols/components/esp_modem/test/host_test/junit.xml files: esp-protocols/components/esp_modem/test/host_test/junit.xml
host_test_gcov_esp_modem: host_test_gcov_esp_modem:
if: contains(github.event.pull_request.labels.*.name, 'modem') if: contains(github.event.pull_request.labels.*.name, 'modem') || github.event_name == 'push'
name: Run gcovr on esp modem host test name: Run gcovr on esp modem host test
runs-on: ubuntu-22.04 runs-on: ubuntu-22.04
container: espressif/idf:release-v4.3 container: espressif/idf:release-v4.3

View File

@ -9,7 +9,7 @@ on:
jobs: jobs:
build_esp_modem_tests: build_esp_modem_tests:
if: contains(github.event.pull_request.labels.*.name, 'modem') if: contains(github.event.pull_request.labels.*.name, 'modem') || github.event_name == 'push'
name: Build Target tests name: Build Target tests
strategy: strategy:
matrix: matrix:
@ -46,7 +46,9 @@ jobs:
target_tests_esp_modem: target_tests_esp_modem:
# Skip running on forks since it won't have access to secrets # Skip running on forks since it won't have access to secrets
if: ${{ contains(github.event.pull_request.labels.*.name, 'modem') && (github.repository == 'espressif/esp-protocols') }} if: |
github.repository == 'espressif/esp-protocols' &&
( contains(github.event.pull_request.labels.*.name, 'modem') || github.event_name == 'push' )
name: Run Target tests name: Run Target tests
strategy: strategy:
matrix: matrix:

View File

@ -9,7 +9,7 @@ on:
jobs: jobs:
build_esp_mqtt_cxx: build_esp_mqtt_cxx:
if: contains(github.event.pull_request.labels.*.name, 'mqtt') if: contains(github.event.pull_request.labels.*.name, 'mqtt') || github.event_name == 'push'
name: Build name: Build
strategy: strategy:
matrix: matrix:

View File

@ -9,7 +9,7 @@ on:
jobs: jobs:
build_websocket: build_websocket:
if: contains(github.event.pull_request.labels.*.name, 'websocket') if: contains(github.event.pull_request.labels.*.name, 'websocket') || github.event_name == 'push'
name: Build name: Build
strategy: strategy:
matrix: matrix:
@ -56,7 +56,9 @@ jobs:
run-target-websocket: run-target-websocket:
# Skip running on forks since it won't have access to secrets # Skip running on forks since it won't have access to secrets
if: ${{ contains(github.event.pull_request.labels.*.name, 'websocket') && (github.repository == 'espressif/esp-protocols') }} if: |
github.repository == 'espressif/esp-protocols' &&
( contains(github.event.pull_request.labels.*.name, 'websocket') || github.event_name == 'push' )
name: Target test name: Target test
needs: build_websocket needs: build_websocket
strategy: strategy:

View File

@ -13,3 +13,19 @@ Contributions in the form of pull requests, issue reports, and feature requests
For quick merging, the contribution should be short, and concentrated on a single feature or topic. The larger the contribution is, the longer it would take to review it and merge it. For quick merging, the contribution should be short, and concentrated on a single feature or topic. The larger the contribution is, the longer it would take to review it and merge it.
Please follow the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) rule when writing commit messages. Please follow the [conventional commits](https://www.conventionalcommits.org/en/v1.0.0/) rule when writing commit messages.
## Release process
When releasing a new component version we have to:
* Update the version number
* Update the changelog
* Create the version tag in this repository
* Deploy the component to component registry
* Update the documentation
This process is not fully automated, the first three steps need to be performed manually by project maintainers running the `bump` command (from within this repository, rather than forks, to publish the release `tag`). Release procedure is as follows:
* Create a branch in this repository (not from fork)
* Run `cz bump [version]` (version number is optional, `cz` would automatically increment it if not present)
* Check the updated `CHANGELOG.md`
* Create and merge the branch to master