Skip to content

Conversation

@alesj
Copy link
Contributor

@alesj alesj commented Dec 12, 2025

Looks like we (really) don't need this dependency, which causes problems here: #51045

@alesj alesj requested review from brunobat and geoand December 12, 2025 16:43
@quarkus-bot

This comment has been minimized.

@quarkus-bot

This comment has been minimized.

@alesj alesj changed the title Remove unneeded vertx-http, upgrade LGTM image, add extensions=true in lgtm tests Remove unneeded vertx-http, upgrade LGTM image Dec 12, 2025
@quarkus-bot
Copy link

quarkus-bot bot commented Dec 12, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit ceb95df.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

You can consult the Develocity build scans.

@quarkus-bot
Copy link

quarkus-bot bot commented Dec 12, 2025

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit ceb95df.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@github-actions
Copy link

🎊 PR Preview 530b821 has been successfully built and deployed to https://quarkus-pr-main-51539-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants