camel
camel copied to clipboard
camel-jbang edit plugin display extra information on camel options
Description
Requires: https://github.com/camel-tooling/camel-language-server/pull/1286 Nice to have but not required: https://github.com/jline/jline3/pull/1327
camel-jbang edit plugin display extra information on camel options like group, required, possible values and default value:
Thanks !
Target
- [ ] I checked that the commit is targeting the correct branch (Camel 4 uses the
mainbranch)
Tracking
- [ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).
Apache Camel coding standards and style
- [ ] I checked that each commit in the pull request has a meaningful subject line and body.
- [ ] I have run
mvn clean install -DskipTestslocally from root folder and I have committed all auto-generated changes.
:star2: Thank you for your contribution to the Apache Camel project! :star2:
:robot: CI automation will test this PR automatically.
:camel: Apache Camel Committers, please review the following items:
-
First-time contributors require MANUAL approval for the GitHub Actions to run
-
You can use the command
/component-test (camel-)component-name1 (camel-)component-name2..to request a test from the test bot. -
You can label PRs using
build-all,build-dependents,skip-testsandtest-dependentsto fine-tune the checks executed by this PR. -
Build and test logs are available in the Summary page. Only Apache Camel committers have access to the summary.
-
:warning: Be careful when sharing logs. Review their contents before sharing them publicly.
a new release of the Language Server with https://github.com/camel-tooling/camel-language-server/pull/1286 is available.
EDIT: I pushed a commit with the change to the branch
@johnpoth the LSP has been released and we have upgraded. Can you maybe get this PR ready. @gnodet have not released jline so maybe we need to another PR when he has done that
@davsclaus your right let's get this merged and merge https://github.com/jline/jline3/pull/1327 later when it's ready
@apupier thank you so much for the update! looks like your commit is already in main branch (dependabot) https://github.com/apache/camel/blob/main/parent/pom.xml#L93
LGTM