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

fix(rest-api-model-server): fix compability #182

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

odzhychko
Copy link
Contributor

@odzhychko odzhychko commented Nov 8, 2023

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.

@odzhychko odzhychko requested a review from a team as a code owner November 8, 2023 08:34
@odzhychko odzhychko force-pushed the fix/ktor-and-quarkus-compatibility branch from ab4e0de to b2adb22 Compare November 8, 2023 08:58
@languitar
Copy link
Contributor

We should bump to Quarkus 3 soon in any case.

Copy link
Contributor

@languitar languitar left a 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.

@nkoester nkoester self-requested a review November 8, 2023 10:15
Copy link
Member

@nkoester nkoester left a 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

@odzhychko odzhychko force-pushed the fix/ktor-and-quarkus-compatibility branch from b2adb22 to b237e56 Compare November 8, 2023 11:03
@odzhychko odzhychko requested a review from languitar November 8, 2023 11:04
@odzhychko odzhychko force-pushed the fix/ktor-and-quarkus-compatibility branch from b237e56 to c736c46 Compare November 8, 2023 11:06
@odzhychko odzhychko requested a review from languitar November 8, 2023 11:07
@odzhychko odzhychko force-pushed the fix/ktor-and-quarkus-compatibility branch from c736c46 to c822139 Compare November 9, 2023 07:32
@odzhychko odzhychko requested a review from nkoester November 9, 2023 07:32
@odzhychko odzhychko merged commit 9606d5d into main Nov 9, 2023
2 checks passed
@odzhychko odzhychko deleted the fix/ktor-and-quarkus-compatibility branch November 9, 2023 08:18
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