Skip to content

Conversation

@apupier
Copy link
Contributor

@apupier apupier commented Dec 16, 2025

Description

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.

@github-actions github-actions bot added the dsl label Dec 16, 2025
@apupier
Copy link
Contributor Author

apupier commented Dec 16, 2025

hum wait, when I build the project locally, the comment is removed from the pom file installed in the .m2 folder

@apupier apupier marked this pull request as draft December 16, 2025 13:46
Cannot use just a comment because it is removed when the pom file is
"installed"

Signed-off-by: Aurélien Pupier <apupier@ibm.com>
@apupier
Copy link
Contributor Author

apupier commented Dec 16, 2025

Given that the comments are not kept, we have to change somethign else. I went to add an unused property. Other preferences?

@apupier apupier requested review from davsclaus and oscerd December 16, 2025 13:50
@apupier apupier marked this pull request as ready for review December 16, 2025 13:51
@oscerd
Copy link
Contributor

oscerd commented Dec 16, 2025

That's ok

@apupier
Copy link
Contributor Author

apupier commented Dec 16, 2025

the new MD5 is no more starting with 070702:
57f486032699c7243be40a1e82d2441e camel-jbang-main/4.14.3/camel-jbang-main-4.14.3.pom

@apupier apupier merged commit b5060cc into apache:release/4.14.3 Dec 16, 2025
2 checks passed
@apupier apupier deleted the 4.14.3-workaround branch December 16, 2025 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants