-
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
Deprecate the quarkus-resteasy-mutiny extension #24005
Deprecate the quarkus-resteasy-mutiny extension #24005
Conversation
.../deployment/src/main/java/io/quarkus/resteasy/mutiny/deployment/ResteasyMutinyProcessor.java
Outdated
Show resolved
Hide resolved
I wonder if we should sneak this one in into 2.7? Maybe worth asking @tqvarnst . |
08553f5
to
212f0dc
Compare
return new FeatureBuildItem(Feature.RESTEASY_MUTINY); | ||
} | ||
|
||
void warn() { | ||
Logger.getLogger(ResteasyMutinyProcessor.class).warn("The quarkus-resteasy-mutiny extension is deprecated. " + |
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.
Maybe it's easier to use io.quarkus.logging.Log.warn
here, but this is fine
@gsmet / @cescoffier there should be a section about this in https://github.com/quarkusio/quarkus/wiki/Migration-Guide-2.8 imho the message from the warning can be used, it's quite nice:
|
Good point, added |
Also added a warning explaining why it's deprecated.