Skip to content

Conversation

@zakkak
Copy link
Collaborator

@zakkak zakkak commented Nov 20, 2023

Resolves NullPointerException introduced with 5103185 (#7827) as reported in quarkusio/quarkus#37212

Closes quarkusio/quarkus#37212

Copy link
Member

@loicottet loicottet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zakkak
Copy link
Collaborator Author

zakkak commented Nov 22, 2023

Thank you for the review @loicottet. Can you also please confirm whether this in the merge queue?

@loicottet
Copy link
Member

@zakkak it should be merged over the weekend

@zakkak
Copy link
Collaborator Author

zakkak commented Nov 24, 2023

Great, thanks for the update.

@jerboaa
Copy link
Collaborator

jerboaa commented Nov 28, 2023

@loicottet Since it's past the weekend now. Did something fail when trying to integrate? This is still causing an issue for us in quarkus tests. Thanks!

@zakkak
Copy link
Collaborator Author

zakkak commented Nov 28, 2023

Merged as #7908

@zakkak zakkak closed this Nov 28, 2023
@zakkak zakkak deleted the 2023-11-20-handle-unspecified-module-name branch November 28, 2023 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement. quarkus quarkus related issue redhat-interest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[GraalVM 24.0] quarkus-integration-test-kotlin-serialization fails with latest GraalVM master tree and quarkus main branch

3 participants