Skip to content

Update futures.md (zh-cn) #1417

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

Closed
wants to merge 2 commits into from
Closed

Update futures.md (zh-cn) #1417

wants to merge 2 commits into from

Conversation

corkine
Copy link

@corkine corkine commented Jul 4, 2019

Updated some areas where the statement is unreasonable, and added some missing translations according to the English documentation.

Updated some areas where the statement is unreasonable, and added some missing translations according to the English documentation.
@NthPortal NthPortal changed the title Update futures.md Update futures.md (zh-cn) Jul 5, 2019
@eed3si9n
Copy link
Member

@corkine Thanks for the contribution!

I can't review this myself so could you find someone who can? /cc @liufengyun, @dongxuwang, @xingyif

@eed3si9n
Copy link
Member

Also #1406 is awaiting review. Maybe you can review his?

@dongxuwang
Copy link
Member

Ok, will do

@dongxuwang dongxuwang self-requested a review July 18, 2019 01:28
Copy link
Member

@dongxuwang dongxuwang left a comment

Choose a reason for hiding this comment

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

In my opinion, some places need to be changed.

@SethTisue
Copy link
Member

@corkine interested in returning to this?

@corkine
Copy link
Author

corkine commented Jul 31, 2019

Sorry, I am a newbie, I will handle this.

fix some translate format problem
Copy link
Author

@corkine corkine left a comment

Choose a reason for hiding this comment

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

comment

@corkine corkine closed this Aug 2, 2019
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.

4 participants