-
Notifications
You must be signed in to change notification settings - Fork 309
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 docs #2180
Fix broken links in docs #2180
Conversation
docs/backends/wrappers/custom.md
Outdated
@@ -277,6 +277,7 @@ class MyCustomBackendHttpTest extends HttpTest[Future] { | |||
} | |||
``` | |||
|
|||
[//]: # (TO REVIEWER last update 6 years ago maybe we have sth newer) |
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.
let's just remove it then
docs/backends/http4s.md
Outdated
@@ -35,6 +35,6 @@ Please note that: | |||
|
|||
Instead, all custom timeout configuration should be done by creating a `org.http4s.client.Client[F]`, using e.g. `org.http4s.client.blaze.BlazeClientBuilder[F]` and passing it to the appropriate method of the `Http4sBackend` object. | |||
|
|||
The backend supports streaming using fs2. For usage details, see the documentation on [streaming using fs2](fs2.md#streaming). | |||
The backend supports streaming using fs2. For usage details, see the documentation on [streaming using fs2](https://sttp.softwaremill.com/en/latest/backends/fs2.html#streaming). |
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 let's just keep a link to fs2.md
to avoid hard-coding links to our own docs
docs/index.md
Outdated
@@ -48,7 +48,10 @@ sttp is a family of Scala HTTP-related projects, and currently includes: | |||
Third party projects: | |||
|
|||
* [sttp-oauth2](https://github.com/ocadotechnology/sttp-oauth2): OAuth2 client library for Scala | |||
|
|||
[//]: # (* TO REVIEWER repo archived by owner) |
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.
let's remove this then
Thanks :) |
closes #1419