-
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
Fix broken links in the RestEasy Reactive guide #35781
Conversation
:resteasy-reactive-common-api: https://javadoc.io/doc/io.quarkus.resteasy.reactive/resteasy-reactive-common/{quarkus-version} | ||
:jsonpapi: https://javadoc.io/doc/jakarta.json/jakarta.json-api/2.1.2/jakarta.json | ||
:injectapi: https://javadoc.io/static/jakarta.inject/jakarta.inject-api/2.0.1/jakarta.inject | ||
:vertxapi: https://javadoc.io/static/io.vertx/vertx-core/4.4.5 |
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.
Hm... Hardcoding the Vert.x version doesn't sounds to great, right?
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.
How is this handled in the Vert.x guide?
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.
I know, but given that the API is unlikely to change (and I was a bit lazy to deal with version filtering) I thought it would be okay for now
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.
How is this handled in the Vert.x guide?
The Vert.x Javadoc is rendered using frames by default, so I linked to the non-framed version. I didn't find any external link to the Vert.x Javadocs in the Vert.x guide
🙈 The PR is closed and the preview is expired. |
384bd8d
to
be1851b
Compare
be1851b
to
0074355
Compare
No description provided.