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

docs: clarify documentation of sync module #6955

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rcoh
Copy link
Contributor

@rcoh rcoh commented Nov 6, 2024

Based on feedback from customers, adding a clearer and top-line comment about runtime-agnosticism for the sync module.

Motivation

Builders were unsure if they could use these primitives outside of Tokio. It is in the docs, but it's near the bottom.

Solution

Added more docs.

Based on feedback from customers, adding a clearer and top-line comment about runtime-agnosticism for the sync module.
@github-actions github-actions bot added the R-loom-sync Run loom sync tests on this PR label Nov 6, 2024
@mox692
Copy link
Member

mox692 commented Nov 11, 2024

I think we got a pretty similar PR a little while ago in #6833. Is that the same as what you're looking for?

@mox692 mox692 added T-docs Topic: documentation A-tokio Area: The main tokio crate labels Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio Area: The main tokio crate R-loom-sync Run loom sync tests on this PR T-docs Topic: documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants