-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Bump smallrye-open-api.version from 4.0.5 to 4.0.6 #45424
Conversation
This comment has been minimized.
This comment has been minimized.
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.
Something is odd. The JDK tests didn’t run. I need to figure out what’s wrong with CI and get another run.
There was some strange behavior in some other repos today as well and I think something mentioned about it on Github status. |
@MikeEdgar thanks for this. Just a note, it does not (yet) fix Fixes #45266, there needs some config to enable this in Quarkus. I'll make a PR for it once this is merged |
It's an issue on our side. I think #45437 might fix it. I will rebase this PR as soon as I can get it in. |
2f943bc
to
62adee1
Compare
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.
Matrix issue is solved. Let's wait for CI and merge.
This comment has been minimized.
This comment has been minimized.
@Ladicek could you have a look at the error above? My guess is that it's not due to this update but it looks suspicious anyway. Also the problem got the build completely stuck:
|
Signed-off-by: Michael Edgar <[email protected]>
62adee1
to
cc15f64
Compare
Status for workflow
|
Fixes #44773
Fixes #45150
Required for #45266