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.

Sign up for GitHub

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

Jump to bottom

Use proper type for serializing Kotlin response #28134

Merged
merged 2 commits into from
Oct 6, 2022
Merged

Use proper type for serializing Kotlin response #28134

merged 2 commits into from
Oct 6, 2022

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Sep 21, 2022

Fixes: #28096

@geoand geoand mentioned this pull request Sep 21, 2022
Closed
@geoand
Copy link
Contributor Author

geoand commented Sep 21, 2022

CI fails on my fork so this obviously is not enough.

@geoand
Copy link
Contributor Author

geoand commented Sep 22, 2022

I pushed an updated version

@geoand geoand marked this pull request as ready for review October 6, 2022 05:54
@quarkus-bot

This comment has been minimized.

Sign in to view
@geoand
Copy link
Contributor Author

geoand commented Oct 6, 2022

This should be good to go now

@geoand geoand requested a review from Sgitario October 6, 2022 07:58
@geoand geoand added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Oct 6, 2022
@quarkus-bot
Copy link

quarkus-bot bot commented Oct 6, 2022

Failing Jobs - Building 26f53a7

Status Name Step Failures Logs Raw logs
✔️ JVM Tests - JDK 11
✖ JVM Tests - JDK 11 Windows Build Failures Logs Raw logs
✔️ JVM Tests - JDK 17
✖ JVM Tests - JDK 17 MacOS M1 Set up runner ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 18

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 Windows #

- Failing: extensions/quartz/deployment

📦 extensions/quartz/deployment

✖ io.quarkus.quartz.test.FireTimeTest.testExecution line 39 - More details - Source on GitHub

java.lang.AssertionError: 

Expecting actual:

@geoand geoand merged commit a18cb69 into quarkusio:main Oct 6, 2022
@geoand geoand deleted the #28096 branch October 6, 2022 11:33
@quarkus-bot quarkus-bot bot added this to the 2.14 - main milestone Oct 6, 2022
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Oct 6, 2022
@gsmet gsmet modified the milestones: 2.14 - main, 2.13.2.Final Oct 10, 2022
@qaltrakadmin qaltrakadmin mentioned this pull request Nov 17, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sgitario Sgitario Sgitario approved these changes

Assignees
No one assigned
Labels
area/rest triage/flaky-test
Projects
None yet
Milestone
2.13.2.Final
Development

Successfully merging this pull request may close these issues.

Serializer for class 'ArrayList' is not found.
3 participants
@geoand @Sgitario @gsmet