- Notifications
You must be signed in to change notification settings - Fork7.8k
fix(ci): Fix IDF examples compilation when changing source files#12001
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
👋Hello lucasssvaz, we appreciate your contribution to this project! 📘 Please review the project'sContributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you haveread and signed theContributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
Memory usage test (comparing PR against master branch)The table below shows the summary of memory usage change (decrease - increase) in bytes and percentage for each target.
Click to expand the detailed deltas report [usage change in BYTES]
| ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
0ae0838 intoespressif:masterUh oh!
There was an error while loading.Please reload this page.
Description of Change
This pull request introduces a targeted enhancement to the logic for determining affected sketches in component mode. The main change ensures that when any source or header file (excluding
.inofiles) within thecores/orlibraries/directories is modified, all IDF component examples are automatically flagged for recompilation. This helps maintain build integrity in response to low-level code changes.Component mode logic improvements:
.inofiles) incores/orlibraries/is changed, ensuring comprehensive coverage for critical component updates.Test Scenarios
Tested locally