Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Quarkus version to 3.9 and update reactive artifacts #353

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

jedla97
Copy link
Member

@jedla97 jedla97 commented Apr 1, 2024

As code.quarkus.io now have default 3.9 it failing to find some artifacts which were renamed.
The replacement guide was https://github.com/quarkusio/quarkus/blob/main/relocations/generaterelocations.java

Also there was change that when generating codestarts index.html is no longer generated quarkusio/quarkus#37680
We testing if it's awailable but to me it's seems just to check if Quarkus start and is able to receive requests

@jedla97 jedla97 marked this pull request as draft April 2, 2024 07:24
@rsvoboda
Copy link
Member

rsvoboda commented Apr 5, 2024

3.9.2 is out, @jedla97 can you update this PR?

@rsvoboda
Copy link
Member

rsvoboda commented Apr 5, 2024

io.quarkus.ts.startstop.ArtifactGeneratorTest/manyReactiveExtensionsSetB/warmup-artifact-build.log

[INFO] --- quarkus-maven-plugin:3.9.1:create (default-cli) @ standalone-pom ---
[WARNING] The extension catalog will be narrowed to the io.quarkus.platform:quarkus-bom:3.9.1 platform release. To enable the complete Quarkiverse extension catalog along with the latest recommended platform releases, please, make sure the extension registry client is enabled.
[ERROR] Cannot find a dependency matching 'quarkus-jaxrs-client', maybe a typo?

@jedla97 jedla97 force-pushed the update-to-3.9 branch 2 times, most recently from 11eb7cb to 269478f Compare April 5, 2024 18:08
@jedla97 jedla97 marked this pull request as ready for review April 7, 2024 22:12
@jedla97 jedla97 requested a review from rsvoboda April 7, 2024 22:21
@fedinskiy
Copy link
Contributor

@rsvoboda could you take a look at this PR? CI is green and these changes are required for #355

@fedinskiy fedinskiy self-requested a review April 10, 2024 11:27
@fedinskiy fedinskiy merged commit 5d1fe71 into quarkus-qe:main Apr 10, 2024
9 checks passed
@fedinskiy
Copy link
Contributor

Thank you for the fix!

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.

None yet

3 participants