Skip to content

Conversation

Jason2866
Copy link

Description:

Related issue (if applicable): fixes #

Checklist:

  • The pull request is done against the latest develop branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR, more changes are allowed when changing boards.json
  • I accept the CLA

* use boards.json settings for HybridCompile

Refactor ESP-IDF configuration handling to support board-specific configurations and improve flash/PSRAM settings management.

* Settings for HybridCompile are choosen now from boards.json

Removed now unnecessary custom SDK configuration options for ESP32S3.

* Fix esp_idf_size command arguments

Remove unnecessary '--ng' argument from esp_idf_size command

* Update esp-idf-size package version to 2.0.0

* Enable HPM features for SPI flash in ESP-IDF

* Add pydantic dependency version to espidf.py

* Remove unused import of join from os.path
Refactor ESP-IDF configuration handling by removing unused functions and adding flash configuration management.
Added a function to extract and normalize the ESP-IDF framework version from the CMake version file and environment variables. Updated linker script preprocessing to support both IDF 5.x and 6.x.
Refactor flash frequency handling and add board-specific configuration checks for sdkconfig generation.
Copy link

coderabbitai bot commented Oct 9, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Jason2866 Jason2866 merged commit 41ac913 into core_dump Oct 9, 2025
67 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.

1 participant