Closed sigmaaa closed 1 month ago
The recent update in the IDFBuildConfiguration
class involves the removal of the configuration
field, substituting it with a local variable within the getProperty
method to streamline the configuration handling process.
File Path | Change Summary |
---|---|
.../idf/core/build/IDFBuildConfiguration.java |
Removed configuration field; added local configuration in getProperty . |
🐰✨ In the land of code, where the bits align, A rabbit hopped through, making changes fine. Fields were plucked, locals placed with care, In hopes that "Hello World" errors would be rare. Hop, skip, a commit, then push, A developer's life, always in a rush. 🌟 🐰💻
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@sigmaaa hi !
Tested under: OS - Windows 10 / MacOS ESP-IDF: v5.2.1
LGTM 👍 Able to build with different "build" folder PATH.
Description
wip.
Fixes # (IEP-1213)
Type of change
Please delete options that are not relevant.
How has this been tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Dependent components impacted by this PR:
Checklist
Summary by CodeRabbit