camel icon indicating copy to clipboard operation
camel copied to clipboard

camel-jbang edit plugin display extra information on camel options

Open johnpoth opened this issue 5 months ago • 1 comments
trafficstars

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: Screenshot from 2025-06-05 11-36-04

Thanks !

Target

  • [ ] I checked that the commit is targeting the correct branch (Camel 4 uses the main branch)

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 -DskipTests locally from root folder and I have committed all auto-generated changes.

johnpoth avatar Jun 10 '25 09:06 johnpoth

: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-tests and test-dependents to 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.

github-actions[bot] avatar Jun 10 '25 09:06 github-actions[bot]

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

apupier avatar Jun 23 '25 13:06 apupier

@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 avatar Jun 25 '25 08:06 davsclaus

@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

johnpoth avatar Jun 25 '25 09:06 johnpoth