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 README link #3447

Merged
merged 2 commits into from
Jun 16, 2024
Merged

Fix README link #3447

merged 2 commits into from
Jun 16, 2024

Conversation

bwignall
Copy link
Contributor

The README currently contains "There are Scaladoc API documentations for [the core library][core-api], which defines and..."; note that a link has not been rendered to "[core-api]".

Looking through git blame, "core-api" previously existed (relevant commit) and pointed to a link on sonatype.org. The reference "[core-api]" changed to "[api]" in this commit but one reference to "core-api" was not updated.

README.md Outdated
@@ -33,7 +33,7 @@ Quick links:

### <a id="docs"></a>Documentation and getting help ###

* There are Scaladoc API documentations for [the core library][core-api], which defines and implements the core types for streams and pulls, as well as the type aliases for pipes and sinks. [The `io` library][io-api] provides FS2 bindings for NIO-based file I/O and TCP/UDP networking.
* There are Scaladoc API documentations for [the core library][api], which defines and implements the core types for streams and pulls, as well as the type aliases for pipes and sinks. [The `io` library][io-api] provides FS2 bindings for NIO-based file I/O and TCP/UDP networking.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually the API docs are no longer split into core/io, it's all under the [api] link now. I suspect the second link is also broken.

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 have removed reference to that split. I made the phrasing of this bullet point similar to the one after it. Please let me know if you have other preferences/suggestions on wording.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As there remained no references to [io-api] or to [rx-api], I removed the Sonatype links to both.

Copy link
Member

@armanbilge armanbilge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you so much!

@armanbilge armanbilge merged commit 2ffe590 into typelevel:main Jun 16, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants