Skip to content
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

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

gastaldi
Copy link
Contributor

@gastaldi gastaldi commented Sep 6, 2023

No description provided.

: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
Copy link
Contributor

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?

Copy link
Contributor

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?

Copy link
Contributor Author

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

Copy link
Contributor Author

@gastaldi gastaldi Sep 6, 2023

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

@github-actions
Copy link

github-actions bot commented Sep 6, 2023

🙈 The PR is closed and the preview is expired.

@gastaldi gastaldi force-pushed the fix_reactive_links branch 2 times, most recently from 384bd8d to be1851b Compare September 6, 2023 15:31
@geoand geoand merged commit 7fa174d into quarkusio:main Sep 8, 2023
@quarkus-bot quarkus-bot bot added this to the 3.5 - main milestone Sep 8, 2023
@gastaldi gastaldi deleted the fix_reactive_links branch September 11, 2023 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

2 participants