Skip to content

Conversation

@fedinskiy
Copy link
Contributor

Partial replacement for #1742
Based on #1792

@fedinskiy fedinskiy requested a review from a team as a code owner September 19, 2025 15:14
@quarkus-bot

This comment has been minimized.

@fedinskiy
Copy link
Contributor Author

fedinskiy commented Sep 19, 2025

According to the failures, I will need to disable the tests by default. Will do this on Monday, and possibly will add an integration test.
For now I need to make sure, that you're not against these changes, @geoand

@geoand
Copy link
Collaborator

geoand commented Sep 19, 2025

It seems to make sense

@fedinskiy fedinskiy force-pushed the chatbot-tests branch 2 times, most recently from 0c98e3f to d4d610e Compare September 22, 2025 07:59
@quarkus-bot

This comment has been minimized.

@fedinskiy
Copy link
Contributor Author

test failures:
codestarts, docs, samples, tetsing-internal, multiple-providers, ollama, rag: Error downloading JBang from https://github.com/jbangdev/jbang/releases/latest/download/jbang.tar

I presume, that there are some problems with jbang site or with respective GH action, and this error is not related to the content of the PR. @geoand could you review the PR?

@geoand
Copy link
Collaborator

geoand commented Sep 22, 2025

Seems reasonable

@quarkus-bot

This comment has been minimized.

@quarkus-bot
Copy link

quarkus-bot bot commented Sep 22, 2025

Status for workflow Build (on pull request)

This is the status report for running Build (on pull request) on commit b1fc04c.

✅ 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.

@geoand geoand merged commit 34170ae into quarkiverse:main Sep 22, 2025
150 of 160 checks passed
@fedinskiy fedinskiy deleted the chatbot-tests branch September 22, 2025 12:29
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.

2 participants