-
Notifications
You must be signed in to change notification settings - Fork 2
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
fix(rest-api-model-server): fix compability #182
Conversation
ab4e0de
to
b2adb22
Compare
We should bump to Quarkus 3 soon in any case. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one idea for a more canonical test structure. Nice corner case catch and explanations.
rest-api-model-server/src/test/kotlin/ReplicatedRepositoryInstantiationTest.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes the previous issues and the quarkus API works with this fix
b2adb22
to
b237e56
Compare
rest-api-model-server/src/test/kotlin/ReplicatedRepositoryInstantiationTest.kt
Outdated
Show resolved
Hide resolved
b237e56
to
c736c46
Compare
c736c46
to
c822139
Compare
See comments in the changed file for details.
The problem is, that we want to use the platform release and that the platform release has broken (for some cases, like our Quarkus case) Ktor version.
The issue arises because of an older Quarkus version, which needs an older Kotlin version.