-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
ci: activate CI for jpa-without-entity integrations native test #5369
ci: activate CI for jpa-without-entity integrations native test #5369
Conversation
Thanks @machi1990 ! |
Would actually be nice if Azure CI ran this :) |
For some reason CI did not pick it :-( |
@machi1990 perhaps you need to also make a dummy "code" change? |
7f5c45e
to
a528d70
Compare
We normally have ci runs for this kind of change. Also, tricking the ci with a code change did not help. |
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.
great. Thanks!
Also the merge of #5343 parent PR is encountering ci failures https://dev.azure.com/quarkus-ci/quarkus/_build/results?buildId=11930&view=logs
something to do with Azure maybe? |
This is based on #5343 (comment)
Fixes #5368
/cc @Sanne @geoand @jaikiran