Closed sigmaaa closed 4 months ago
The changes across the project involve updating package names from com.espressif.idf.lsp
to com.espressif.idf.core
and reorganizing imports accordingly. Additionally, functionality enhancements include updating the ClangdConfigFileHandler
to incorporate the project's build folder name and modifying methods in IDFBuildConfiguration
for better console output and build handling.
File Path | Change Summary |
---|---|
.../src/com/espressif/idf/core/ILSPConstants.java |
Updated package name and modified CLANGD_CONFIG_FILE declaration. |
.../src/com/espressif/idf/core/build/IDFBuildConfiguration.java |
Added imports for GCCToolChain and ClangdConfigFileHandler , updated build method and writeConsoleNotes method. |
.../src/com/espressif/idf/core/util/ClangdConfigFileHandler.java |
Updated package name, added imports, and modified updateCompileFlagsSection method. |
.../src/com/espressif/idf/lsp/preferences/IDFClangdOptionsDefaults.java |
Updated import statement for ILSPConstants . |
.../src/com/espressif/idf/ui/handlers/ClangdConfigCreateHandler.java |
Updated import statement for ClangdConfigFileHandler . |
.../src/com/espressif/idf/ui/wizard/NewIDFProjectWizard.java |
Added and removed import statements for ClangdConfigFileHandler . |
In the land of code where constants shift, Packages change and imports lift. Build configurations now refined, Clangd flags with folders aligned. A rabbit hops with joy, you see, For code's as clean as it can be! 🐇✨
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 ESP-IDF: v5.2.1
Create project -> build with Custom Build folder -> all headers are resolved -> ✔️ Create project -> build with Custom Build folder -> all headers are resolved -> "Project Full Clean" -> re-build -> all headers are resolved ✔️
LGTM 👍
Description
Fixed unresolved headers after build with custom build folder by editing clang file after the build. Also improved clangd Troubleshooting messages after the build
Fixes # (IEP-1212)
Type of change
Please delete options that are not relevant.
How has this been tested?
Test Configuration:
Dependent components impacted by this PR:
Checklist
Summary by CodeRabbit
Refactor
com.espressif.idf.lsp
tocom.espressif.idf.core
for better organization and consistency.build
method to include a new handler for updating project configurations.New Features
CompileFlags
section with the project's build folder name, ensuring more accurate configurations.Chores