IEP-1656 Update LSP4E to the 0.29.0
Description
Please include a summary of the change and which issue is fixed.
Fixes # (IEP-1656)
Type of change
Please delete options that are not relevant.
- New feature (non-breaking change which adds functionality)
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 A
- Test B
Test Configuration:
- ESP-IDF Version:
- OS (Windows,Linux and macOS):
Dependent components impacted by this PR:
- Component 1
- Component 2
Checklist
- [ ] PR Self Reviewed
- [ ] Applied Code formatting
- [ ] Added Documentation
- [ ] Added Unit Test
- [ ] Verified on all platforms - Windows,Linux and macOS
Summary by CodeRabbit
- Chores
- Updated Language Server Protocol support to the latest version, enhancing IDE integration and development environment stability.
Walkthrough
Updated the InstallableUnit version for LSP4E from 0.27.10 to 0.29.0 in the target definition file. The change only alters the source/version of the LSP4E bundle; other LSP4E/LSP4J units and locations remain unchanged.
Changes
| Cohort / File(s) | Summary |
|---|---|
LSP4E Version Bump releng/com.espressif.idf.target/com.espressif.idf.target.target |
Updated InstallableUnit for LSP4E from version 0.27.10 to 0.29.0 (source/version change only) |
Estimated code review effort
π― 2 (Simple) | β±οΈ ~10 minutes
- Verify compatibility of LSP4E 0.29.0 with existing LSP4E/LSP4J dependencies and build.
- Spot-check target resolution and plugin startup with updated target.
Possibly related PRs
- espressif/idf-eclipse-plugin#1317 β Previously set LSP4E InstallableUnit to 0.27.10; directly related target-definition changes.
- espressif/idf-eclipse-plugin#1245 β Prior update to LSP4E in the same target definition (related file-level edits).
Suggested reviewers
- alirana01
- kolipakakondal
Poem
π° I hopped from .10 to .29 with glee,
A tiny bump for LSP, a spring in me.
Bundles updated, target sings,
Quiet changes, hopeful springs. β¨
Pre-merge checks and finishing touches
β Passed checks (3 passed)
| Check name | Status | Explanation |
|---|---|---|
| Description Check | β Passed | Check skipped - CodeRabbitβs high-level summary is enabled. |
| Title check | β Passed | The title 'IEP-1656 Update LSP4E to the 0.29.0' directly and clearly describes the main change: updating LSP4E dependency to version 0.29.0, which matches the raw summary and PR objectives. |
| Docstring Coverage | β Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
β¨ Finishing touches
π§ͺ Generate unit tests (beta)
- [ ] Create PR with unit tests
- [ ] Post copyable unit tests in a comment
- [ ] Commit unit tests in branch
IEP-1656
π Recent review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
π₯ Commits
Reviewing files that changed from the base of the PR and between 768f2f79f32819e6a4d0124458f84b73f1198174 and 25e65d04e43490ef9453a8e717d19a5bc437682c.
π Files selected for processing (1)
releng/com.espressif.idf.target/com.espressif.idf.target.target(1 hunks)
π§ Files skipped from review as they are similar to previous changes (1)
- releng/com.espressif.idf.target/com.espressif.idf.target.target
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
Hi @kolipakakondal, @AndriiFilippov,
With this PR, the new option now appears under the Language Server settings:
This option fixes the new-line indentation issue on Linux and macOS.
@sigmaaa hi !
Tested under: OS: Windows 11 / Mac ARM64 / Linux Ubuntu
the new-line indentation issue fixed :heavy_check_mark:
LGTM :+1: