Skip to content

Conversation

geoand
Copy link
Collaborator

@geoand geoand commented Aug 8, 2025

No description provided.

@geoand geoand requested a review from a team as a code owner August 8, 2025 16:31
@geoand
Copy link
Collaborator Author

geoand commented Aug 8, 2025

cc @gsmet @aloubyansky since they are interested in the platform

@geoand geoand merged commit 76d4625 into main Aug 8, 2025
2 checks passed
@geoand geoand deleted the release-1.1.1 branch August 8, 2025 16:32
@aloubyansky
Copy link
Member

If this is what we want in the platform then update this property https://github.com/quarkusio/quarkus-platform/blob/main/pom.xml#L86, run mvn -Dsync (may take some time on the first run), add all the changes and open a PR. Thanks!

@geoand
Copy link
Collaborator Author

geoand commented Aug 8, 2025

Will do next week, thanks!

@geoand
Copy link
Collaborator Author

geoand commented Aug 8, 2025

@fedinskiy the release failed with:

2025-08-08T16:35:52.2595753Z [INFO] ------------------------------------------------------------------------
2025-08-08T16:35:52.2956490Z [ERROR] Failed to execute goal org.apache.maven.plugins:maven-release-plugin:3.1.1:prepare (default-cli) on project quarkus-langchain4j-parent: The artifact (io.quarkiverse.langchain4j:quarkus-langchain4j-anthropic) requires a different version (1.1.1) than what is found (${project.version}) for the expression (${quarkus.langchain4j.version}) in the project (io.quarkiverse.langchain4j:quarkus-langchain4j-integration-test-anthropic). -> [Help 1]
2025-08-08T16:35:52.2959597Z [ERROR] 

which I assume is related to a0a37f8.

As I'm on PTO and don't have time to look into this, can you please take a look?

@fedinskiy
Copy link
Contributor

@geoand I didn't expect such behavior from maven release plugin :(

Created #1687, it fixed the issue locally, so let's hope it will work in CI as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants