feat: update lsp4e dependency - #1500
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe target definition updates the LSP4E p2 repository URL from release ChangesLSP4E repository update
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized dependency update is supported by target-platform resolution checks and clangd workflow testing; no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Description
Bump the target-platform LSP4E p2 repo from 0.30.5 to 0.30.8.
Type of change
How has this been tested?
org.eclipse.lsp4e/lsp4jfrom 0.30.8IDFProjectLspCdtFunctionalityTest,NewEspressifIDFProjectClangFilesTestmain.c— clangd start, content assist, hover, diagnostics, format, inlay hints, outline, signature help.cfile:--compile-commands-dir=still applied and clangd restartsTest Configuration:
Dependent components impacted by this PR:
org.eclipse.lsp4e/org.eclipse.lsp4e.debug/org.eclipse.lsp4j*com.espressif.idf.lsp,LspService)Checklist
Summary by CodeRabbit