fix(ci): Cleanup and extension of workflows [v0.18] #61
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
roma-jam
force-pushed
the
fix/override_tinyusb_in_network_example
branch
from
c9d27bf to
a101b49
Compare
roma-jam
changed the title
roma-jam
force-pushed
the
fix/override_tinyusb_in_network_example
branch
from
a101b49 to
a740eb6
Compare
roma-jam
changed the title
roma-jam
changed the title
roma-jam
force-pushed
the
fix/override_tinyusb_in_network_example
branch
3 times, most recently
from
c526ace to
4a85b18
Compare
roma-jam
changed the title
There was a problem hiding this comment.
Pull Request Overview
This PR enhances and fixes CI workflows for ESP-IDF integration, focusing on build and test automation. The changes centralize version management, expand target coverage, and improve artifact handling.
Key changes:
- Centralized tool versions (
idf-component-managerandidf-build-apps) as environment variables for easier maintenance - Extended build targets from esp32s2-only to all supported targets, while keeping test execution limited to esp32s2
- Introduced configuration file for filtering and configuring ESP-IDF example builds
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
.github/workflows/build_iot_examples.yml |
Added version environment variables, renamed paths for clarity, fixed component override for network example |
.github/workflows/build_and_run_idf_examples.yml |
Added version variables, configuration file support, expanded build targets, improved artifact path handling |
.github/workflows/build_and_run_esp_usb_test_apps.yml |
Added version variables and workflow description, expanded to all targets, improved path handling and documentation |
.github/ci/.build_and_run_idf_examples_config.toml |
New configuration file to exclude cherryusb examples and standardize build directory naming |
- Add all targets to esp-usb test apps and esp-idf examples - Fix override tinyusb in the network example
roma-jam
force-pushed
the
fix/override_tinyusb_in_network_example
branch
from
4a85b18 to
8486e5c
Compare
peter-marcisovsky
approved these changes
Oct 22, 2025
igi540
approved these changes
Oct 22, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix and extension for previously added CI workflows:
Common
idf-component-managerandidf-build-apps) as environment variablesIDF_COMP_MAN_VERandIDF_BUILD_APPS_VERto simplify the version update procedure in the future.Tools might be updated, so to decrease the time of searching the necessary value in the workflow code, they moved to
env.ESP-USB Test Apps - Build & Run
ESP-IDF USB Device examples - Build & Run
.github/ci/.build_and_run_idf_examples_config.toml), to exclude unnecessary examples (e.g. cherryusb) and configure the idf-build-apps.ESP-IDF Network example (sta2eth) - Build