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

Update README.md #510

Merged
merged 6 commits into from
May 23, 2024
Merged

Update README.md #510

merged 6 commits into from
May 23, 2024

Conversation

SimonsMine
Copy link
Contributor

Added note about windows

@SimonsMine SimonsMine requested a review from a team as a code owner January 20, 2024 16:26
Copy link
Collaborator

@hi-rustin hi-rustin left a comment

Choose a reason for hiding this comment

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

image

It seemed you lost a ) here.

README.md Outdated Show resolved Hide resolved
@SimonsMine
Copy link
Contributor Author

fixed : )

Copy link
Member

@hawkw hawkw left a comment

Choose a reason for hiding this comment

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

Thanks for helping to improve the documentation!

I suggested some edits to the note on UTF-8 support, let me know what you think?

Also, would you mind adding a similar note to the tokio-console crate's README, in the "running the console" section? That documentation is rendered on https://crates.io/crates/tokio-console and https://docs.rs/tokio-console, so it would be nice to include the Windows support information there as well.

README.md Outdated Show resolved Hide resolved
@SimonsMine
Copy link
Contributor Author

Applied suggestions : )

@hawkw hawkw enabled auto-merge (squash) February 6, 2024 17:27
Copy link
Collaborator

@hi-rustin hi-rustin left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks! 👍

Signed-off-by: hi-rustin <[email protected]>
@hawkw hawkw merged commit a0d20fd into tokio-rs:main May 23, 2024
17 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.

None yet

3 participants