-
Notifications
You must be signed in to change notification settings - Fork 195
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
Upgrade Quarkus to 3.4.0.CR1 #5270
Conversation
6ddb118
to
8e540b3
Compare
For reasons that I don't quite understand, I have to add |
Shouldn't be that bad anyway: knative is more an internal thing if I remember well ? |
It's a transitive of The weird thing is that the error only occurs when the build is run from the project root. And not when run directly from the |
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.
Ok clearer, a difference between top project and sub module could be the parent pom interpolation that would bring something in the mix... not sure it's a good track.
Given that knative is not meant for direct usage (from what I remind), the workaround might not be be that bad.
@jamesnetherton I think this is because of the But I don't know if there is a specific reason for that import. |
This reverts commit de45fab.
…ptaplanner extensions apache#5277
4abf12e
to
c5ade29
Compare
I am expecting some test failures. Lets see how it goes...