Skip to content

Conversation

@PetrESP
Copy link
Collaborator

@PetrESP PetrESP commented Nov 14, 2025

ESP-BSP Pull Request checklist

  • CI passing

Change description

Aligned all BSPs and components to have overridden paths


Note

Standardizes idf_component.yml files to use local override_path for component dependencies across BSPs/components and updates READMEs/benchmarks and the PR template.

  • BSP configurations (idf_component.yml):
    • Add override_path for local deps across multiple BSPs: esp_lcd_touch_*, esp_lcd_ili9341, esp_lcd_gc9503, esp_lcd_ili9881c, esp_lcd_lt8912b, esp_lvgl_port, icm42670, and various sensors (hts221, bh1750, fbm320, mag3110, mpu6050).
    • Adjust dependency declarations and examples in: bsp/esp-box-3, bsp/esp-box, bsp/esp32_azure_iot_kit, bsp/esp32_p4_eye, bsp/esp32_p4_function_ev_board, bsp/esp32_s3_korvo_2, bsp/esp32_s3_lcd_ev_board, bsp/esp_bsp_generic, bsp/m5stack_core_2, bsp/m5stack_core_s3.
  • Component configs (components/*/idf_component.yml):
    • Add override_path to point to local bases for esp_io_expander and esp_lcd_touch families in: io_expander/*, lcd_touch/*.
    • Minor YAML fix in esp_io_expander_tca9554 (public field formatting).
  • Docs:
    • Update .github/PULL_REQUEST_TEMPLATE/pr_template_bsp.md checklist to require url and overridden paths.
    • Refresh README benchmarks (dates, LVGL version/metrics) for esp-box-3, esp32_s3_lcd_ev_board, m5stack_core_s3; minor table tweak in esp32_p4_function_ev_board.

Written by Cursor Bugbot for commit b97b4ec. This will update automatically on new commits. Configure here.

@github-actions
Copy link

github-actions bot commented Nov 14, 2025

Test Results

28 tests   28 ✅  1m 30s ⏱️
 5 suites   0 💤
 5 files     0 ❌

Results for commit b97b4ec.

♻️ This comment has been updated with latest results.

@PetrESP PetrESP force-pushed the petr/components/override_paths branch 5 times, most recently from 1c34822 to 27d6cc8 Compare November 20, 2025 13:45
@github-actions
Copy link

⚠️ Build failed for idf_ver=release-v6.0 index=5

This failure was ignored (continue-on-error enabled).
See details: View build logs

@PetrESP PetrESP force-pushed the petr/components/override_paths branch from 27d6cc8 to 015e577 Compare November 20, 2025 14:14
@PetrESP PetrESP marked this pull request as ready for review November 20, 2025 15:12
@PetrESP PetrESP requested a review from espzav November 20, 2025 15:12
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@PetrESP PetrESP force-pushed the petr/components/override_paths branch from 015e577 to f109165 Compare November 20, 2025 15:34
@PetrESP PetrESP force-pushed the petr/components/override_paths branch from f109165 to b97b4ec Compare November 21, 2025 08:09
@github-actions
Copy link

⚠️ Build failed for idf_ver=latest index=5

This failure was ignored (continue-on-error enabled).
See details: View build logs

@github-actions
Copy link

⚠️ Build failed for idf_ver=latest index=4

This failure was ignored (continue-on-error enabled).
See details: View build logs

@github-actions
Copy link

⚠️ Build failed for idf_ver=latest index=3

This failure was ignored (continue-on-error enabled).
See details: View build logs

Copy link
Collaborator

@espzav espzav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@PetrESP Thank you for changes! LGTM

@PetrESP PetrESP merged commit c0e8ffa into master Nov 21, 2025
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants